Repository navigation
chore(sync): merge upstream dev for 2.2.6-shuv.2 - #479
Merged
Merged
Conversation
Co-authored-by: dc85 <83497+dc85@users.noreply.github.com> Co-authored-by: Daniel Chen <daniel85.chen@gmail.com>
…page, and fix WebKit's reload jump (anomalyco#62445)
Co-authored-by: Daniel <daniel@anoma.ly>
Co-authored-by: Daniel <daniel@anoma.ly>
…he cli as optional (anomalyco#62494)
Co-authored-by: Daniel <daniel@anoma.ly>
…yco#62831) Co-authored-by: Ryan Skidmore <4580448+ryanskidmore@users.noreply.github.com>
…alyco#62926) Co-authored-by: Ryan Skidmore <4580448+ryanskidmore@users.noreply.github.com>
…2937) Co-authored-by: Frank <frank@anoma.ly>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
This was referenced Nov 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
chore(sync): merge upstream dev through a5fda7e
Base:
integration-v2· Head:sync/upstream-dev-20261109Summary
Merge 140 upstream
devcommits after the #477 pin, preserve the existing Shuvcode overlay, and prepare2.2.6-shuv.2. This PR does not merge itself, publish, or deploy anything.Includes #478 (Claude CLI identity). After #478 landed,
origin/integration-v2was merged normally inde46a5bace; no rebase or history rewrite.packages/claude-pluginis byte-for-byte identical to the merged #478 tree. All verification below ran after that merge.SHAs
6a581a34c564da7c93cc8a1b19060c49811cafa951cede01da3c9ae590222d62f7aec06a53acea9e64f2403ffa55b8bbbd7c05f79b630f866f0ae66adevpin, mergeda5fda7ee1347eb3bb1227724e0ad8cdc091ad7e320c056d84249fe29be7244ff4f9e0aad044c4799de46a5bacec2008616e703102c6e425d8a1d177f715fd38f27dba3bfcbf7b75269c0ca41d55446cfCommits after
de46a5bacechange only documentation/changelog, test fixtures/preload and the hygiene allowlist, not product code, dependencies, generated clients, or build configuration. The checks/results below distinguish those test-only fixes from the earlier builds.Commits (first parent)
0eb60e5f14docs(sync): inventory preserved fork branches and freeze dev pinbd6aebb7dachore(sync): merge upstream dev through a5fda7e73abd1f7dcchore(release): prepare 2.2.6-shuv.2 notes and version floorde46a5bacenormal merge oforigin/integration-v2, including fix(claude-plugin): follow installed Claude CLI identity #478e1d92e04f3docs(sync): record merged Claude CLI identity update55ac7ee6d2test(core): supply config for location activity tool flagsd201eed3fatest(app): await composer focus before opening reviewfccce0789edocs(sync): record completion audit and validationWhat the merge brings
Fork preservation
shuvcodepackage/bin, Shuvcode ACP identity, XDG app id, client service fallback/spawn identity, default ports0x1337/ local0x1338.shuvcode/shuvcode-node; CLI-only publish ownership guard forshuv1337/shuvcode.$skill, quick quote, subagent sidebar and Night Owl default.isHiddenProvider.runwithautostart: false, fix(core): inherit parent agent denies on child sessions #475 fail-closed parent-agent deny inheritance, redirect guards, and chore(sync): merge upstream dev and audit v2.2.6 gaps #477's router-matched cross-site guard.Conflict resolutions were checked with
git show --remerge-diff bd6aebb7da(12 files), not GitHub's capped diff endpoint. This PR exceeds 300 files: review a localgit diff origin/integration-v2...HEADafter fetching the branch.The September rewrite map is not an exact November inventory: the standalone Codex identity plugin and xAI popular-order override were already absent at the #477 base. Night Owl now lives in the theme default. These historical differences are documented rather than silently restoring old implementations or claiming they exist.
Branch inventory
The complete inventory and completion audit records a disposition for every local/origin unmerged branch at the original base, excluding upstream refs and the symbolic default-branch alias. It has 188 grouped rows:
No branch was deleted or automatically reconciled. No additional missing fix was identified. Historical merged PRs document their original landing, not permission to restore pre-rewrite code. #478 is included only through the requested normal merge.
Upstream changes to review
These touch request handling, permissions, auth or execution boundaries. Listed for the reviewer; this completion pass does not supply a new security assessment:
Version:
2.2.6-shuv.2CLI manifest and lockfile carry
2.2.6-shuv.2;CHANGELOG.mdcontains its section, upstream pin, #478 and carried known issues. Release version resolution now honors the checked-in minimum when npm latest trails it.OPENCODE_CHANNEL=latestwas checked and resolves2.2.6-shuv.2.The requested
integration-v2builds intentionally use the existing preview-version convention, not release versions: Bun printsshuvcode v0.0.0-integration-v2-202611092217; Node printsshuvcode-node v0.0.0-integration-v2-202611092220. No release-channel artifact is claimed here.Verification
Every runtime check used throwaway HOME/XDG/database state, no inherited credentials, and a private loopback-only network namespace. The namespace retained the normal unprivileged uid, not root, so permission tests remained meaningful. No running user service, config or session was used.
bun run checktypecheck:tscloop from fork CIjjunavailable); all 0 fail.packages/clientbun run generate: exit 0, no generated changes.bun run script/publish.ts --dry-run: exit 0; Node and workerd-condition import probes pass. No publishing occurs.OPENCODE_CHANNEL=integration-v2 bun run --cwd packages/cli build: pass, all configured targets.build:node --single --skip-install, reusing the Bun build's embedded web archive: pass.packages/apptypecheck:e2e51cede01daarchive; exact diagnostic messages match after ignoring shifted line numbers. Not a new regression; canonical and CI typechecks pass.Fixes found by verification
55ac7ee6d2: upstream addedFlag → ConfigtoTool.node; the fork's location-activity test graph intentionally supplies its own Location. ProvideConfig.testLayer()so the real Tool/Flag graph can be compiled. Before: import-timeUnbound layer node: @opencode/Location; after: all 20 cases pass, including 10 repeats each. No product code change.d201eed3fa: wait for initial composer autofocus before opening review in the editor-launch timeout test. Upstream now preserves focus changes during panel animation. Before: 3/10 targeted failures at the panel-focus assertion; baseline51cede01dapassed 10/10. After: 10/10 pass, and the full session-header keeper 28/28 pass. All existing focus, launch, disabled-state and timeout assertions remain; no sleeps, retries or product changes were added.E2E
Final full suite on
d201eed3fa: 755 passed, 0 failed, five workers, no retries (4.3 minutes). Command:PLAYWRIGHT_BUILD=1 PLAYWRIGHT_BASE_URL=http://127.0.0.1:4310 bun run test:e2e, using a fresh private static server and the existing production-fixture build. The two earlier full runs usedbun --cwd packages/app test:e2e:built; no app product code changed between the build and the final run.Earlier full runs (both before the last e2e test correction) were 754/755 and 753/755. The repeated focus failure is addressed above. The second run also intermittently failed mobile touch-tab reorder; that unchanged test passed 10/10 in isolated repeats on both this branch and the
51cede01dabaseline. That evidence alone does not establish a root cause.Deferred / unverified
next/betaversions remains deferred. Those implementations are unchanged.20c056d842is deliberately not merged.Feature flag
No new fork flag: this is an upstream sync, release metadata and test-fixture fixes. Upstream's experimental
codemode_vmremains controlled by the existing flag registry (on for local/dev, off for release/other preview channels).Review fixes
/Users/meexample without editing upstreamread.ts.auth.json.bun run check39/39 tasks passed, hygiene PII/artifact check clean, AGENTS/preload Prettier andgit diff --checkpassed. Full isolated core suite: 7,416 passed, 16 skipped, 0 failed in a private loopback namespace with uid 1000 and all capabilities dropped (host port 8080 is occupied). Earlier attempts exposed the old isolation assertion, host-port conflict and namespace capability effects; the final run resolves those conditions without changing product code.b155801676,5024ba68c5,64249b7563,d8b762dc00,55dd16b680,715fd38f27.gh pr checks 479 --repo shuv1337/shuvcode --watchcompleted successfully: all five checks green (hygiene, typecheck, plugin-browser and both Socket checks). No merge or release.