Skip to content

fix(release): include publication helper in recovery checkout - #123

Merged
francoischalifour merged 1 commit into
mainfrom
fc/fix-release-orchestration-import
Sep 29, 2026
Merged

francoischalifour merged 1 commit into
mainfrom
fc/fix-release-orchestration-import

Conversation

@francoischalifour

Copy link
Copy Markdown
Member

The v1.8.0 recovery run reached attestation but failed before asset upload: its sparse checkout contained publish-release.ts without the shared isRecord helper that the script imports.

Include cli/src/lib/object.ts in that checkout and use a relative import so the publication script resolves without the CLI TypeScript path alias. Add a test that copies only the declared sparse-checkout files into an isolated directory and builds the script. It reproduced the recovery failure before the fix and passes afterward.

Verification: ./scripts/verify.sh passed (762 CLI tests, 79 black-box tests, 40 browser tests). The v1.8.0 release still has no assets; after this PR lands, its recovery workflow can be dispatched again.

@francoischalifour
francoischalifour merged commit 2a7b2dc into main Sep 29, 2026
12 checks passed
@francoischalifour
francoischalifour deleted the fc/fix-release-orchestration-import branch September 29, 2026 17:04
francoischalifour pushed a commit that referenced this pull request Oct 1, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.9.0](v1.8.0...v1.9.0)
(2026-10-01)


### Features

* **apps:** package the data app runtime as an npm dependency
([#124](#124))
([bd807c1](bd807c1))


### Bug Fixes

* **release:** include publication helper in recovery checkout
([#123](#123))
([2a7b2dc](2a7b2dc))
* **release:** recover assetless v1.8.0 release
([#121](#121))
([bdf930f](bdf930f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants