Skip to content

fix: separate host and skill write targets before locking - #1510

Closed
ScriptedAlchemy wants to merge 4 commits into
codex/tracedecay-total-redesign-plan-reopenedfrom
cursor/separate-before-serializing-shared-state-0673
Closed

ScriptedAlchemy wants to merge 4 commits into
codex/tracedecay-total-redesign-plan-reopenedfrom
cursor/separate-before-serializing-shared-state-0673

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Sep 17, 2026 •

Copy link
Copy Markdown
Owner

Summary

Motivation

Principle: separate-before-serializing-shared-state.

This PR stays on the #707 tip (codex/tracedecay-total-redesign-plan-reopened). It does not merge #707 or this PR to master.

Merge

  • Tip SHA merged: 3e6167b265639c17b7d1c174cd3b7baa3116adae
  • Conflicts: none
  • Tip already absorbed NotApplicable admission and the removed changes.reused list. Terminal publication park tests still pass.

Changes

  • HostBundleWriterV1 no longer takes lifecycle-root writer.v1.lock or host-blind journal.v1.json. Journals are journal.{slug}.v1.json; locks are writer.{slug}.v1.lock. Legacy names are still read and retired only for the host that wrote them.
  • Skill usage writers update one skill file under agent_managed/skill_usage/<hex>.json.
  • read_publication_pointer identifies a cached pointer by content digest, not size and mtime.

Test plan

  • cargo test -p tracedecay-code-index-runtime --lib publication_store (27 passed, including the undecoded CAS refusal)
  • cargo test -p tracedecay-code-index-runtime --lib branch_publication (10 passed, including terminal publication park)
  • cargo test -p tracedecay-code-index-runtime --lib park_convergence
  • cargo test -p tracedecay-agent-hosts --lib host_bundle::tests (33 passed)
  • cargo test -p tracedecay-automation-runtime --lib recording_one_skill_does_not_rewrite_another_skills_file
  • cargo nextest run --workspace --no-fail-fast not run

Checklist

  • CHANGELOG.md updated
  • No secrets, credentials, or .env files included
  • Breaking changes documented (if any)
Open in Web Open in Cursor 

cursoragent and others added 2 commits September 17, 2026 01:39
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
@changeset-bot

changeset-bot Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: b85d942

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

cursoragent and others added 2 commits September 17, 2026 01:48
…r/separate-before-serializing-shared-state-0673

Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

Integrated or superseded directly on the #707 branch at ff36c4f; follow-up compile/typecheck regressions were fixed on that branch.

@ScriptedAlchemy
ScriptedAlchemy deleted the cursor/separate-before-serializing-shared-state-0673 branch October 11, 2026 00:05
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.

2 participants