refactor(dowel): tier site-work into its boundary and its procedure #997
No reviewers
Labels
No labels
move-to-repo
coilyco-bridge-deploy
move-to-repo
coilyco-flight-deck-agent-compose
move-to-repo
coilyco-gaming-eco-app
move-to-repo
coilysiren-inbox
move-to-repo
unknown
🔒⚠️📦⚠️🔒 SANDBOXED 🔒⚠️📦⚠️🔒
autonomy
async-consult
autonomy
epic
autonomy
headless
autonomy
live-collab
c#
priority
P0
priority
P1
priority
P2
priority
P3
priority
P4
role/ai
role/creator
role/design
role/director
role/engineer
role/exec
role/human
role/ops
role/qa
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
coilyco-gaming/sirens-echo!997
Loading…
Reference in a new issue
No description provided.
Delete branch "task/dowel-site-work-tier"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Kai's call on #993, taken one file at a time. This is the self-contained half.
site-work.mdwas one 4,566-byte file markedinline: always, so every turnpaid for all of it whether or not a page was being touched. It holds two kinds
of content, and #927's rule separates them.
What stays inline
The half that must never be missed:
entire landing workflow
winning where they disagree
size not being a reason to defer
If that were deferred, Dowel could hand a site edit to an operator who does not
exist, which is the exact
boundary-modify-live-systemdrifttooling-sirens-dowel-contractwas written to prevent.What moves out
site-work-procedure.md, deferred: what to do with what a write returned,working in visible increments, and the changelog rules. Only meaningful once a
verb is actually being called.
site-work.mdnow points at it in place of themoved text.
Smaller than I estimated, and worth saying
I told Kai roughly 3.5KB would move. 1,434 bytes actually does, because the
ownership section alone is 1,715 and has to stay.
So the honest benefit is that the tier now matches the content, not that the
prompt got much lighter. The whole file was already inline, so this does not
improve safety either. It is a modest, zero-risk trim.
No deploy change, deliberately
References are discovered by walking the root, so a second file in an existing
root needs no
local_skill_rootsedit. That matters: coilyco-bridge/deploy#666named a root the image did not carry and crashlooped the lane for 90 minutes
today. This change cannot repeat that.
Not in this PR
Kai also asked for
moxn-knowledge-base.mdandtemporal.mdto becomestandalone skills, deferred. That needs new skill roots, which deploy's
local_skill_rootsmust name, which is exactly the multi-repo sequencing thatcaused today's outage. It has to go image-first and gets its own PR.
coilyco-suite.mdstays inline, unchanged, per Kai.Verification
go test ./...andpre-commitpass. Split checked section by section againstthe original, with the ownership boundary confirmed present in the inline half.
Refs #993, #968, #927