Skip to content

Conversation

@seefeldb
Copy link
Contributor

@seefeldb seefeldb commented Oct 23, 2025

Summary by cubic

Updated the DX1 plan to use TypeScript branded types for capabilities and switch to an in-place migration that unifies the Cell API. Graph snapshots move to Phase 2, and the pattern integration test harness is complete; the V2 opt-in plan is removed.

  • Refactors

    • Replaced proxy-based capability wrappers with branded types: CellLike plus OpaqueRef, Cell, ReadonlyCell, WriteonlyCell, and Stream. OpaqueRef keeps proxy behavior.
    • Cause derivation now has two layers: automatic defaults and explicit .for(cause, flexible?) with clear throw rules.
    • Graph snapshot work deferred to Phase 2 and aligned to process metadata.
    • Overview updated to in-place migration, deferred execution, and cell creation without immediate links.
    • Integration harness marked complete; dual-mode removed. Deleted implementation-plan-v2.md.
  • Migration

    • Rollout plan adds concrete tasks: unify Cell types, disable ShadowRef/unsafe_, support deferred cell creation and .for(), track causes, and write process metadata.
    • Run recipes like lifts with tracked cell/cause creation; drop JSON recipe representation.
    • Add AST .for(variableName, true) and update schemas.
    • Cleanup: remove legacy OpaqueRef and .setDefault. Future work: ship serializable node factories (nodeFactory@1, .curry).

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 issues found across 7 files

Prompt for AI agents (all 2 issues)

Understand the root cause of the following 2 issues and fix them.


<file name="docs/specs/recipe-construction/rollout-plan.md">

<violation number="1" location="docs/specs/recipe-construction/rollout-plan.md:65">
The plan item misspells the intended helper as `Cell.for(casue)`, which will confuse readers about the correct API name and parameter order.</violation>
</file>

<file name="docs/specs/recipe-construction/graph-snapshot.md">

<violation number="1" location="docs/specs/recipe-construction/graph-snapshot.md:116">
This rewording conflicts with the earlier Snapshot Envelope section, which still directs persisting the snapshot under the `graph` metadata key; please reconcile the document to name a single field.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

@seefeldb seefeldb merged commit 78cf029 into main Oct 24, 2025
8 checks passed
@seefeldb seefeldb deleted the plan/updated-dx1-plan branch October 24, 2025 00:52
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