Repository navigation
fix(fork): integrate checkpoint A security and provider updates - #13
Merged
unlockdigital merged 11 commits intoNov 4, 2026
Merged
unlockdigital merged 11 commits into
unlockdigital merged 11 commits into
Conversation
Backport upstream ad4bb6d and dcc31b8. Verify the Chrome macOS arm64 archive independently. Apply compatible node-forge, shell-quote, smol-toml, path-to-regexp and dbus-next XML fixes. Preserve the existing fork patches and add a multi-document resolved-lockfile security check to CI and release builds. Workspace audit findings remain separate from reference-only Dependabot alerts.
Prepare the rollback checkpoint for reference credentials from upstream 84537c4, bff68fb and a104909, Usage filters from e6e2ab0, and once schedules from 4ac97be. Keep supported settings and private routing intact, retain opaque references and Usage filters on disk, suppress stale credential materialization, and reject replacement of unsupported credentials. Omit unsupported schedules from lists while retaining their rows and existing non-execution behavior. This introduces no migrations.
Reject normalized host collisions before rewriting saved references or processing settings updates. Canonicalize credential maps before validating protected slots, and prevent protected secrets from entering materialization for each cached or emitted settings snapshot. Regression fixtures fail before the fix for aliased updates and both saved-key orders. Preserve rejected settings byte for byte and verify stale secret-store entries are never read.
Backport the upstream leading-bound checks into the pinned Claude SDK bundle and verify the installed bundle digest. Preserve check ordering, custom errors and asynchronous parsing with bounded regression coverage. Upstream: colinhacks/zod a0ebf970f2d752e5f58527cd2942ff48be4f46bd (GHSA-mrhx-c7xc-m3w5). Scope: embedded Zod v4 array path; legacy v3 and unconstrained collections retain separate qualification.
Use pnpm directly for the critical audit threshold because the Vite+ package-manager adapter rejects audit-level. The tested command retains all lower-severity findings in its output.
Exercise the current continuation prompt after interrupted actions and read-only checks. Grade target-specific verification before a retry and preserve fake conversation history. The production instruction text remains unchanged; real-model baseline and candidate evaluation are deferred.
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
The combined PDF regression reached the normal 15-second limit at different steps on loaded CI runners. Split revision, fit-width, and path-reset coverage into focused cases while retaining the assertions and nonzero horizontal-scroll precondition.
The fork completed all 5,295 web tests and the production boot check, but dependency setup plus validation exceeded the job’s ten-minute ceiling. Allow twenty minutes on fork runners while retaining upstream’s job budget and existing individual-test deadlines.
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.
This brings the approved checkpoint-A upstream fixes into the Unlock Digital fork while preserving its dependency patches, account routing, privacy controls, and migration bridge. It updates bundled browsers, provider modes and Codex recovery, diagnostic redaction, and delegation/Claude Stop behavior. Forward-read safeguards preserve newer credential-reference settings and unsupported schedules during rollback.
The Claude SDK remains pinned with a reviewed backport for its embedded Zod v4 leading-bound array checks. The dependency guard validates resolved security floors and the installed SDK bundle hash. CI uses native pnpm for the advisory threshold. The fork web job has a twenty-minute budget for dependency setup, the full suite and production boot; individual test deadlines stay at their existing values.
PDF reload coverage runs as three focused scenarios. It preserves the page, zoom, scroll offsets, fit-width mode and reset on another file path within the normal test budget.
Validation: checkpoint-wide changed-file checks pass, including 1,519 tests across 28 files and typechecks for 39 affected packages. The final test-only correction check passes all 135 tests in the affected file, format, lint, knip and affected typechecks. Frozen installation and macOS arm64 packaging pass. Packaged server files match the build; the package retains the Unlock Digital preview identity and protocol. The two credential-collision findings were independently reviewed and corrected. Full PR CI passed on bc4d5ae, including all 5,295 web tests and production boot: https://github.com/unlockdigital/t3code/actions/runs/41916926254. The final thread-transfer report remains within all enforced ceilings.
Qualification: six workspace advisories remain documented separately from the dismissed reference-only alerts. Legacy SDK Zod v3 reachability remains unresolved. The production restart prompt is unchanged; only offline eval preparation is included. Native visual verification was stopped at the user's command-line-only direction. Current live application data and the original dirty checkout are preserved. Checkpoint B and managed client policies remain outside this change.
Implemented with Codex (GPT-6.1 Sol), with scoped Luna, Terra, and Sol implementation and independent reviews.