fix(ci): restore scripts/publish_skills.sh — second half of the cleanup deletion #132

Closed

mik-tf wants to merge 1 commit from development_mik_ci_publish_sh into development

mik-tf commented

2026-04-25 20:47:38 +00:00

Owner

Follow-up to #131. The same cleanup commit (ce18f5f) that deleted scripts/test.sh also deleted scripts/publish_skills.sh. #131 fixed the test step; the post-merge push run then exposed that the workflow's Publish step (.forgejo/workflows/build.yaml:62) still references the missing publish script.

Restored from git history at 1400fb2^ — same content, no edits. The script:

sources buildenv.sh for SKILLS_ARCHIVE
locates the rolling latest release on Forge (creates if missing)
removes existing same-named asset
uploads skills.tgz as the new asset

Validation

bash -n scripts/publish_skills.sh parses cleanly
This PR's pull_request CI run will exercise build + test (Publish gated to push events per #131)
Post-merge push run will exercise Publish for real — first end-to-end green CI on this repo since February

This + #131 close the long-running CI break.

Signed-off-by: mik-tf

Follow-up to [#131](https://forge.ourworld.tf/lhumina_code/hero_skills/pulls/131). The same `cleanup` commit ([`ce18f5f`](https://forge.ourworld.tf/lhumina_code/hero_skills/commit/ce18f5f)) that deleted `scripts/test.sh` also deleted `scripts/publish_skills.sh`. #131 fixed the test step; the post-merge push run then exposed that the workflow's Publish step (`.forgejo/workflows/build.yaml:62`) still references the missing publish script. Restored from git history at `1400fb2^` — same content, no edits. The script: - sources `buildenv.sh` for `SKILLS_ARCHIVE` - locates the rolling `latest` release on Forge (creates if missing) - removes existing same-named asset - uploads `skills.tgz` as the new asset ## Validation - `bash -n scripts/publish_skills.sh` parses cleanly - This PR's `pull_request` CI run will exercise build + test (Publish gated to push events per #131) - Post-merge push run will exercise Publish for real — first end-to-end green CI on this repo since February This + #131 close the long-running CI break. Signed-off-by: mik-tf

mik-tf added 1 commit

2026-04-25 20:47:38 +00:00

fix(ci): restore scripts/publish_skills.sh — second half of the cleanup deletion

Build and Publish Skills / build-and-publish (pull_request) Successful in 3s

Details

aa5da76351

PR #131 (#131)
restored scripts/test.sh and added pull_request CI triggers; the post-
merge run on development then exposed that scripts/publish_skills.sh
was deleted by the same 'cleanup' commit ce18f5f and the workflow's
Publish step references it (.forgejo/workflows/build.yaml:62).

Restored from history at git rev 1400fb2^ — same content, no edits.
The script:
- sources buildenv.sh for SKILLS_ARCHIVE name
- locates the rolling 'latest' release on Forge (creates if missing)
- removes existing same-named asset
- uploads skills.tgz as the new asset

Validation: bash -n parses cleanly.  Cannot run end-to-end locally
without a release token — CI's pull_request run on this PR will
exercise build + test (publish gated to push), and the post-merge
push run will exercise publish for real.

This + #131 close the long-running CI break introduced by ce18f5f.