docs: apply DevGuard design revision 1 to the integration docs (CSP-D04) - #73
Merged
Merged
Conversation
Update the five counterpart documents that DevGuard design revision 1 (merged as d4981b4 in DevGuard PR #8) lists for CodeSpace, with their reviewed Korean counterparts and registry hashes: - devguard-integration: immutable design references at d4981b4, the CS-RG work order with CSRG-C00/P0 and the mandatory C09 decision, CSP-D04 as the delivery condition before CSRG-P0, the revised 48/25 totals, and the dated DevGuard main kept apart from design provenance. Two statements that contradicted DevGuard ADR-001 and ADR-006 are corrected: DG-1 has no service-exec path, and extending the pinned PTY wrapper cannot carry a managed launch. - architecture: current pipe (Tokio) and PTY (pinned Codex adapter) ownership versus the planned coordination layer, and BackendReaped versus OwnerControlledReap. - execution-substrate: the planned one-time preparation, separate lifetimes and the F1a-F1e protection rules. - codex-reuse: the reuse finding, D1-D3, the ProcessDriver criteria, the limited adaptation rules and the kept pin. - upstream-update: review of any future adaptation on pin changes, and the boundaries for future crates and backends, citing the current dependency and CI selection rules. No product root or component is added for code that does not exist. Planned behavior is labelled as target, not current. No runtime code, configuration, CI policy or pin changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The DevGuard roadmap named the authorized DevGuard source by an absolute path on one development machine. Describe it as a local checkout of the DevGuard repository instead, in English and Korean, and record the pair. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
5 tasks
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.
Summary
This is CSP-D04, the CodeSpace counterpart documentation that DevGuard design revision 1 §14.2 requires. The revision was merged in DevGuard PR #8 as
d4981b4c241cff42687f5c2c681b583c7847776e. DevGuard's delivery plan places CSP-D04 before CSRG-P0.The PR changes documentation only: five documents, their Korean counterparts and their registry hashes. It changes no runtime code, configuration, CI policy, dependency rules or Codex pin. Base
b6e7ed2.ec58668: the CSP-D04 content.23d424e: describes the DevGuard source without an absolute local path that the roadmap had carried sincefb822fc.Changes
Every planned element is labelled as the target of CS-RG. None of it is presented as current behavior or API.
docs/ko)devguard-integration.mdd4981b4. CS-RG work order: CSRG-C00/P0 added, C09 a mandatory convergence decision, C08 on the head C09 leaves. CSP-D04 is a delivery condition before CSRG-P0. Totals are 48 units in 25 groups. DevGuard main (30b5fa6, after PR #9) is recorded with its date, apart from design provenance and from release0.1.0-5daee5d-b3fa569e. Two statements corrected (see below)architecture.mdcodespace-ptyover the pinned Codex adapter. Planned common coordination layer and narrow backend boundary.BackendReapedversusOwnerControlledReap, with noExitedUnreapedcapability from a backend that already reapedexecution-substrate.mdPreparedExecution/LaunchPlan, separate lifetimes, and the F1a–F1e protection rules, including the rule that callers never holdspawn_guardaroundhelper_command()/HelperCommand::spawn()codex-reuse.mdProcessDriveradoption criteria, limited-adaptation rules, and the pin kept by revision 1upstream-update.mdThe two corrected statements had contradicted DevGuard's accepted decisions:
service-execprepares credentials and startup" is superseded wording in ADR-001; DG-1 usesCredentialHandoff.Dependency and CI boundaries
PRODUCTSmust equal the workspace members, andcomponentsmust equal thecrates/directories.upstream_dependencies.py:PRODUCTS,ADAPTERS,FORBIDDENandRUNNER_FORBIDDEN.ci-policy.json:componentsand thefulltrigger list.scripts/tests/test_ci_plan.py.check-no-model-deps.sh.third_party/**,.gitmodules,.github/**,scripts/**anddocs/upstream-lock.mdalready run every leg. New code inside an existing crate is covered by that crate's component, for examplecrates/runner/**.PRODUCTS/ADAPTERS, the adapter allowlist, the CI component andcompileslists, and its tests together. It never narrows the forbidden lists or any other check.Verification
Run locally at
23d424e(macOS 27.0, Python 3.14.7, Node 24.21.0 and npm 11.19.0 asdocs-sitepins them):python3 -B scripts/check_docs.pypython3 -B -m unittest discover -s scripts/testsTMPDIRinside the worktree because of a local sandbox limit)python3 scripts/ci_plan.py --base origin/mainaffected, 0 Rust legs, as expected for documentation-only changesnpm ci/npm test/npm run build --prefix docs-sitepython3 -B docs-site/scripts/site.py checkgit diff --check; scan for<outside code and for private pathsThe first
npm ciunder Node 26.8.1 was refused byengine-strict, as the pinned engines require; the retry used 24.21.0.This PR's CI is the documentation workflow plus a Rust CI plan that selects no legs. Its results are read once and reported here after they finish.
Limitations and rollback