Skip to content

Pin website.change with the organic system - #411

Merged
demegire merged 1 commit into
mainfrom
fix/organic-pins-website-change
Oct 13, 2026
Merged

demegire merged 1 commit into
mainfrom
fix/organic-pins-website-change

Conversation

@demegire

Copy link
Copy Markdown
Contributor

What and why

Since v6, the organic system's technical and delivery steps run website.change (organic_system.WEBSITE_STEPS), but PARENT_CHILD_KEYS never pinned it with the system's revision. Production's registry hid the gap: it builds up over time, and earlier commits already held workflows/website.change.json.

On staging's fresh registry, Claw Messenger's first organic system run (8710f7ff) failed in organic_system_prepare. code.storage returned 404 for workflows/website.change.json at the system's revision. A new self-hosted install would hit the same failure. In production it also means organic runs read whatever website.change happened to be in the registry at that commit, not one deliberately pinned with the system.

Verification

  • test_the_organic_system_pins_every_workflow_its_prepare_step_reads (new): every key the prepare step reads (steps, website steps, measurement steps, refresh) is in pinned_children.
  • The first-sync test now checks the website and measurement children's files at the parent's first revision too.
  • Both fail without the fix and pass with it. Organic system and onboarding tests pass.

Tested on staging

Deployed to staging by pushing to staging (deploy-staging). I'll retry Claw Messenger's organic traffic system there and update this PR with the result.

Notes

On merge, the catalog sync publishes a new revision of organic.traffic_system that includes website.change. Runs and schedules already pinned to the old revision keep it.

🤖 Generated with Claude Code

Since v6 the organic system's technical and delivery steps run website.change, but
the system's revision did not carry its definition. Production's registry hid the gap
because earlier commits already held the file; a fresh registry (staging, a new
install) has no website.change at the system's revision, and every organic system
run failed while preparing (code.storage 404).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@demegire
demegire requested a review from a team as a code owner October 13, 2026 14:15
@demegire

Copy link
Copy Markdown
Contributor Author

Tested on staging: with this branch deployed (deploy-staging run 38411751299), Claw Messenger's organic traffic system (run d4e796d7) gets past organic_system_prepare. The audit and keyword-plan children are running, and the step list shows website.change for technical and delivery. On the previous build, run 8710f7ff failed in prepare with a code.storage 404 for workflows/website.change.json.

@demegire
demegire merged commit 3d1e746 into main Oct 13, 2026
10 checks passed

This branch was successfully deployed

1 active deployment
staging — c0d37dea Deployed Oct 13, 2026 by demegire via deploy #6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant