chore(deps): bump actions/setup-node from 4.4.0 to 6.3.0#53
chore(deps): bump actions/setup-node from 4.4.0 to 6.3.0#53dependabot[bot] wants to merge 16 commits into
Conversation
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 4.4.0 to 6.3.0. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v4.4.0...v6.3.0) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: 6.3.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 84c95f1274
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Co-authored-by: Codex <noreply@openai.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a3032953ac
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0215c24401
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Co-authored-by: Codex <noreply@openai.com>
|
A newer version of actions/setup-node exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged. |
Merge current main into the Dependabot setup-node upgrade and carry forward the repository hook fixes required by the current pre-push governance gate. Validation: DOCS_LINT_SCOPE=staged pnpm docs:lint Validation: pnpm lint Validation: pnpm typecheck Validation: pnpm test Validation: pnpm build Validation: make audit Validation: bash scripts/check-environment.sh Co-authored-by: Codex <noreply@openai.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Match the GitHub check job installation path so npm run check does not invoke Corepack/pnpm during CI. Validation: npm run check Co-authored-by: Codex <noreply@openai.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5651fba089
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| if [[ -n "${CI_BASE_SHA:-}" && -n "${CI_HEAD_SHA:-}" ]]; then | ||
| mapfile -d '' changed_markdown < <( | ||
| git diff --name-only -z "$CI_BASE_SHA" "$CI_HEAD_SHA" -- '*.md' |
There was a problem hiding this comment.
Exclude deleted Markdown files from CI docs lint
The CI path collects files with git diff --name-only but does not apply --diff-filter=ACMR, so deleted .md files are still passed to markdownlint-cli2. On PRs that remove documentation files, this makes the linter receive paths that no longer exist and can fail the docs gate even though there is nothing left to lint; the staged path below already uses --diff-filter=ACMR, so CI and local behavior are inconsistent.
Useful? React with 👍 / 👎.
| git -C "$REPO_ROOT" restore --worktree -- \ | ||
| .diagram/context/diagram-context.md \ | ||
| .diagram/context/diagram-context.meta.json \ | ||
| .diagram/manifest.json |
There was a problem hiding this comment.
Avoid restoring tracked diagram files unconditionally
This unconditional git restore --worktree resets tracked diagram artifacts whenever the check passes, which can silently discard a developer’s unstaged edits during make hooks-pre-push (this script is invoked from that hook). The freshness check should not mutate user worktrees unless it first proves those files were untouched before running the refresh.
Useful? React with 👍 / 👎.
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 102d0c10b7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } catch { | ||
| return undefined; |
There was a problem hiding this comment.
Fail docs-gate when git diff cannot be computed
runGit converts every git error into undefined, and getChangedMarkdownFiles then treats that as an empty change set. In CI or shallow-history contexts where CI_BASE_SHA/CI_HEAD_SHA (or merge-base resolution) are invalid, docs-gate will report success instead of failing, so markdown governance checks are silently bypassed rather than surfaced as an actionable error.
Useful? React with 👍 / 👎.
| echo "Environment check passed (attestation: $ATTESTATION_PATH)" | ||
|
|
||
| if git -C "$REPO_ROOT" ls-files --error-unmatch "${ATTESTATION_PATH#$REPO_ROOT/}" >/dev/null 2>&1; then | ||
| git -C "$REPO_ROOT" restore --worktree -- "${ATTESTATION_PATH#$REPO_ROOT/}" |
There was a problem hiding this comment.
Preserve local edits before restoring attestation file
This unconditional restore drops any unstaged edits to a tracked attestation file after a successful run. Because hooks-pre-push invokes scripts/check-environment.sh, a normal git push can silently erase local work in artifacts/policy/environment-attestation.json; the script should either verify the file was clean before running or fail with guidance instead of mutating user state.
Useful? React with 👍 / 👎.
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Co-authored-by: Codex <noreply@openai.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Resolve workflow, docs gate, and package conflicts while preserving the setup-node upgrade and explicit CLI test-client gate. Co-authored-by: Codex <noreply@openai.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Summary
Automated dependency/workflow update for
jscraik/rSearchPR #53. Scope is limited to the version bump in this branch.Checklist
Testing
Review artifacts
Notes
This body was updated to satisfy repository PR template requirements so checks can evaluate the actual code/workflow changes.