Skip to content

chore(deps): openai 7 — the floor it needs is already under it - #5

Closed
catomean wants to merge 1 commit into
mainfrom
chore/openai-7
Closed

chore(deps): openai 7 — the floor it needs is already under it#5
catomean wants to merge 1 commit into
mainfrom
chore/openai-7

Conversation

@catomean

@catomean catomean commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Closes the fleet ratchet's last counted gap. The sweep held this believing prod ran the EOL system Node 20; the gateway actually runs its own nvm Node v22.22.3 — already over openai v7's only breaking change (Node ≥ 22 floor) — and the box's system Node is 24 now regardless. Production installs from npm-global, so this fork manifest change cannot touch the live gateway.

CI on this PR is the verification (the earlier #2 'no checks' was my premature merge, not absent CI). The security-audit lane is expected red from upstream's own transitive advisories — pre-existing on main, unrelated.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WqKqMnHQHSmkGFfc5t7Rxn

The fleet currency sweep held this bump believing production ran the EOL
system Node 20; inspection showed the gateway runs its own nvm Node
v22.22.3, which already satisfies openai v7's only breaking change (the
Node >= 22 floor). The box's system Node is 24 now regardless. This closes
the fleet version-currency ratchet's last counted gap.

Lockfile-only resolution locally; this fork's CI validates the tree (its
pull_request lanes run — the earlier 'no checks' read on PR #2 was checks
not yet initialized, not absent CI). The security-audit lane is expected
red from upstream's own transitive advisories, pre-existing on main.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WqKqMnHQHSmkGFfc5t7Rxn
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Dependency Guard

This PR changes dependency-related files. Maintainers should confirm these changes are intentional.

Changed files:

  • package.json
  • pnpm-lock.yaml

Maintainer follow-up:

  • Review whether the dependency changes are intentional.
  • Inspect resolved package deltas when lockfile, shrinkwrap, or workspace dependency policy changes are present.
  • Treat package-lock.json and npm-shrinkwrap.json diffs as security-review surfaces.
  • Run pnpm deps:changes:report -- --base-ref origin/main --markdown /tmp/dependency-changes.md --json /tmp/dependency-changes.json locally for detailed release-style evidence.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Dependency graph changes noted

This PR includes dependency graph changes. The dependency guard is informational because the PR author is a repository admin or a member of @openclaw/openclaw-secops.

  • Current SHA: fc4cde89a9cae893c116d3d181a9a7b571a41ba6
  • Trusted actor: @catomean
  • Trusted role: pull request author; repository admin

Security review is still recommended before merge when the dependency graph change is intentional.

@catomean

catomean commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator Author

Closing with a verdict rather than merging red: the bump itself is sound (gateway runs nvm Node 22.22.3 ≥ v7's floor; prod installs from npm-global, untouched by this manifest), but check-test-types fails on this PR with implicit-any/overload errors in the fork's own test files (session-write-lock.test.ts, commands-export-session.test.ts) — fork-baseline breakage (likely @types/node-26 interaction) that must be fixed before any dependency PR here can be CI-verified. Four more lanes (build-artifacts, check-shrinkwrap, checks-fast-bun-launcher, security-fast) are red on main itself. The fleet version-currency ratchet keeps this gap counted (baseline 1) until the fork's CI baseline is repaired.

@catomean catomean closed this Sep 1, 2026
@catomean
catomean deleted the chore/openai-7 branch September 1, 2026 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant