Skip to content

[codex] Harden TaskBound action install path#6

Merged
Conalh merged 1 commit into
mainfrom
codey/taskbound-action-hardening
May 21, 2026
Merged

[codex] Harden TaskBound action install path#6
Conalh merged 1 commit into
mainfrom
codey/taskbound-action-hardening

Conversation

@Conalh
Copy link
Copy Markdown
Owner

@Conalh Conalh commented May 21, 2026

Summary

Hardens the Action publishing path after the live demo showed the self-dogfood workflow could execute PR-modified npm lifecycle scripts when using uses: ./.

Changes:

  • Run npm ci --ignore-scripts and npm run build --ignore-scripts inside the composite Action install/build steps.
  • Switch the self-dogfood workflow from PR-modifiable uses: ./ to Conalh/TaskBound@main, while keeping PR title/body task fallback coverage.
  • Bump package metadata and README examples to 0.2.0 so the next release tag can match the advertised Action behavior.
  • Add workflow tests covering ignored install scripts and the trusted dogfood Action ref.

Root cause

PR #1 intentionally modifies package.json with a postinstall pipe-to-shell script. The self-dogfood workflow used uses: ./, so GitHub Actions evaluated the local Action from the PR merge checkout and ran npm ci from that checkout. The log showed the demo postinstall command executing before TaskBound reviewed the diff.

Validation

  • node --test test/workflow.test.mjs passed.
  • npm run build passed.
  • npm test passed: 22/22.
  • git diff --check passed, with Windows CRLF conversion warnings only.

@Conalh Conalh marked this pull request as ready for review May 21, 2026 22:53
@Conalh Conalh merged commit 07ec31c into main May 21, 2026
2 checks passed
@Conalh Conalh deleted the codey/taskbound-action-hardening branch May 21, 2026 22:53
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.

1 participant