ci: isolate RC release index cleanup

Breadway commented

2026-10-01 15:24:27 +08:00

Owner

Child of ECO-56. The current .forgejo/workflows/rc-release.yml removes /tmp/bread-ecosystem-ci-* before creating its mktemp checkout. That wildcard can delete a sibling workflow's live clone. Replace it with a prefix older workflows cannot match and clean only this invocation's clone on EXIT. Verify successful and failed beta index generation preserve a sibling checkout under the legacy prefix for breadclip.

Child of ECO-56. The current `.forgejo/workflows/rc-release.yml` removes `/tmp/bread-ecosystem-ci-*` before creating its mktemp checkout. That wildcard can delete a sibling workflow's live clone. Replace it with a prefix older workflows cannot match and clean only this invocation's clone on EXIT. Verify successful and failed beta index generation preserve a sibling checkout under the legacy prefix for breadclip.

Breadway added the

ci

label

2026-10-01 15:24:28 +08:00

Breadway referenced this issue from a pull request that will close it,

2026-10-01 15:31:13 +08:00

ci: isolate RC release index cleanup #18

Breadway commented

2026-10-01 15:31:37 +08:00

Author

Owner

PR: #18

The beta index step now uses a prefix older wildcard cleaners cannot match and removes only its own clone via EXIT trap. YAML parsing and simulated success/failure checks passed while a legacy-prefix sibling checkout remained intact. Forgejo CI is running; the issue stays in DOING until it succeeds.

PR: https://git.breadway.dev/Breadway/breadclip/pulls/18 The beta index step now uses a prefix older wildcard cleaners cannot match and removes only its own clone via EXIT trap. YAML parsing and simulated success/failure checks passed while a legacy-prefix sibling checkout remained intact. Forgejo CI is running; the issue stays in DOING until it succeeds.

Breadway commented

2026-10-01 15:37:13 +08:00

Author

Owner

The source change in PR #18 is scoped correctly: rc-release.yml:55-58 uses a unique clone path outside the legacy wildcard prefix and an EXIT trap that removes only this invocation's checkout. The simulated success and failure checks are recorded in the PR description. Current CI for head 619857791e111fa1776f78562995acb83dd0426a is still pending (check / check (push) is waiting to run). Leave this issue in TODO until CI finishes, then request independent review again.

The source change in PR #18 is scoped correctly: `rc-release.yml:55-58` uses a unique clone path outside the legacy wildcard prefix and an EXIT trap that removes only this invocation's checkout. The simulated success and failure checks are recorded in the PR description. Current CI for head `619857791e111fa1776f78562995acb83dd0426a` is still pending (`check / check (push)` is waiting to run). Leave this issue in TODO until CI finishes, then request independent review again.

Breadway commented

2026-10-01 16:21:21 +08:00

Author

Owner

PR #18 passes local YAML and shell checks plus success/failure cleanup simulations with a legacy-prefix sibling preserved. Forgejo CI was queued at last check. Moving to REVIEW for independent review; if CI fails, return it to TODO with the failure recorded.

Breadway commented

2026-10-01 16:26:13 +08:00

Author

Owner

Forgejo CI for PR #18 passed in 6m51s on head 619857791e111fa1776f78562995acb83dd0426a. The isolated success/failure cleanup checks and sibling legacy-prefix preservation check pass. The linked PR is ready for the next workflow stage.

Forgejo CI for PR #18 passed in 6m51s on head `619857791e111fa1776f78562995acb83dd0426a`. The isolated success/failure cleanup checks and sibling legacy-prefix preservation check pass. The linked PR is ready for the next workflow stage.

Breadway closed this issue

2026-10-02 12:04:28 +08:00

Rows
Columns

ci: isolate RC release index cleanup #17