Repository navigation
refactor: use the outer engineering workspace - #238
Merged
kinqsradio merged 6 commits intoOct 25, 2026
Merged
Conversation
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.
Change
Use the outer Orbyn Engineering workspace for engineering policy and skills. The target application no longer carries a copied installation or public-skill pin. This follows the approved nested-only direction; existing third-party targets keep their own instructions and skills.
scripts/ci_scope.py, retaining the complete MIT notice. Retire the obsolete skills lane; docs/full classification, conservative unknown-path handling and deployment trust checks remain.Existing work
Product documentation/MIT/contact PR #231 was reconciled, source-checked and merged first at beb060d. This branch includes that actual merged base; its product changes are not duplicated in this cleanup's diff. Five old draft application stacks remain untouched because historical checks do not establish source equivalence or merge readiness.
Checks and limits
At eec865c: 24 focused disposable CI, safe-server-preflight, source-integrity and workspace-boundary fixtures pass. Static source checks pass for 5,018 files and five gitlinks; local link targets pass for 22 current guides. Changed authored documents are formatted, Python parses and whitespace checks pass. Application/runtime/tool snapshot diff is empty.
The owning lead reviewed the initial cleanup and boundary/template delta. Following the human’s clarified future-project-guidance scope, the final guard allows target-owned instructions/skills/settings and limits rejection to known retired installation controls and raw research. Negative-ignore and bare raw-research-root safeguards remain. The owning lead approved this five-file scope correction on the exact head. The first local retry for this correction encountered seven ENOSPC environment errors; after space recovered without task deletions, all 24 fixtures passed. The failed attempt and successful retry are retained. Hosted CI must pass on this exact final head. CI changes require the full application lane. No local app build, native provider, REA engine, client/model setup or deployment ran. Dated work records retain historical paths and are excluded from current-guide link validation.
The active legacy primary checkout has not moved: live processes still use it. Its untracked audit documents, output and private configuration remain owned there. A later fresh independent nested clone can be prepared under the public workspace's ignored repo/Orbyn, with repo/orbyn-wt task worktrees; this does not transfer private files, databases or process ownership. Nesting is not a claim of live host policy/skill discovery.
Workspace
Engineering changes are coordinated between the public-workspace owner and Orbyn Builder, using the existing relevant guidance and project source. Public policy/tooling changes merged separately through PRs #19 and #20. This candidate changes only Orbyn's target boundary. The REA engine/runtime upgrade remains deferred.