[hawkeyexl]

Summary

The last planned v1 release. It keeps the floating v1 tag on CLI 4, and it stops a non-zero exit from losing results. Closes #80.

• Exit handling: runs with ignoreReturnCode, deletes any stale output file first, and still loads results. The results, issue, and PR steps now run before a non-zero exit fails the job. A missing results file still fails the job right away.
• exit_on_fail: true keeps failing through setFailed. The action.yml default is gone, so empty means false, with a core.notice that v2 defaults it to true.
• version: defaults to 4. It refuses latest, and any version or range that can install 5 or later, such as 5, ^5, >=4, *, and 4 || 5, with a pointer to @v2. Empty still means the locally linked CLI.
• $RESULTS: cut to fit GitHub's 65,536-character limit, with a link to the run. Tokens now fill in one pass, so $PROMPT or $& inside the results stays literal and can't grow the body after it's measured.
• Diagnostics: crash messages now include stderr, since CLI 5 logs there.
• Dispatch workflows: main.yml and test-current-release.yml defaulted version to latest, which would now be refused, so they default to 4.

Decisions to check

• Other dist-tags, like next, get a warning instead of a refusal, because the Action can't tell what they install without a registry call. Tags that read as a version, like v5-rc, are refused.
• The range check is hand-written, to avoid adding semver for one release. v2 deletes it. A bound like <5.0.0-rc.1 refuses, which errs safe.

Skipped

• Annotations and the annotations input. Not trivial.
• Merging the Dependabot majors (Bump @actions/exec from 1.1.1 to 3.0.0 #63, Bump @actions/github from 6.0.1 to 9.1.1 #65, Bump @actions/core from 1.11.1 to 3.0.1 #66). package.json is already on @actions/core 3, @actions/exec 3, and @actions/github 9.
• Merging the job summary PR (feat: attach Doc Detective's markdown reporter output to the job summary #81). It's in the plan, but not in this change's scope.
• Paths with spaces stay broken on v1, since the command is still a string. v2 fixes this.

Testing

• npm test: 36 pass, 0 fail. npm run typecheck passes, and dist/ is rebuilt.
• Ran the built dist/index.js on Windows with a stub npx, covering exit 0, 1, and 2 with exit_on_fail true, false, and unset, with and without failed specs, a stale output file, and refused versions. Each case gave the expected job result and results output.
• Not run: the Actions workflows on hosted runners, or a real issue or PR creation.

🤖 Generated with Claude Code

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn