Skip to content

fix: adopt pnpm 11.6.0 and the consolidated pr-review workflow#25

Merged
Jercik merged 3 commits into
mainfrom
fix/align-managed-files
Jun 13, 2026
Merged

fix: adopt pnpm 11.6.0 and the consolidated pr-review workflow#25
Jercik merged 3 commits into
mainfrom
fix/align-managed-files

Conversation

@Jercik

@Jercik Jercik commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Applies the v1.31.0 alignment wave:

  • packageManager bumped to pnpm@11.6.0
  • Consolidated pr-review.yml (pull_request_target + workflow_dispatch) replaces pr-review-on-open.yml / pr-review-on-push.yml
  • .githooks/commit-msg now invokes conventional-commit-msg via npm exec --package= instead of npx -y, so an inherited npm_config_package (set by agent wrappers like axexec/axrun) can no longer hijack resolution and exit 127 on every commit (template fix: j4k-align #33)

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary: No actionable issues found. Verified the consolidation is behavior-preserving and dependencies resolve. 2 info-level notes.


Code review by Code Review 2 (Claude Code Opus) (opus)

Comment thread .github/workflows/pr-review.yml
Comment thread .githooks/commit-msg

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary: No issues found.


Code review by Code Review 1 (gpt-5.5)

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary: Clean infra-alignment PR (workflow consolidation preserves prior trigger behavior, pnpm bump is consistent with pnpm/action-setup@v5 reading packageManager). Found 1 low-severity observation.


Code review by Code Review 2 (Claude Code Opus) (opus)

Comment thread .githooks/commit-msg
@Jercik Jercik merged commit 4c613cd into main Jun 13, 2026
6 of 8 checks passed
@Jercik Jercik deleted the fix/align-managed-files branch June 13, 2026 06:08
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