Skip to content

Conversation

@seefeldb
Copy link
Contributor

@seefeldb seefeldb commented Oct 24, 2025

Summary by cubic

Expanded the rollout-plan spec to detail merging OpaqueRef with RegularCell and to unify the cell type system. This simplifies cell APIs, removes the need for toOpaqueRef, and supports cells without initial links.

  • Refactors
    • Introduced AnyCell, OpaqueCell, ComparableCell, ReadonlyCell, and WriteonlyCell; OpaqueRef is now a variant of OpaqueCell.
    • Updated CellLike to allow nested AnyCell; Opaque accepts nested CellLike.
    • Simplified wrap/unwrap types to strip AnyCell in inputs and wrap outputs in OpaqueRef.
    • Enabled creating cells without a link; added .for to set a cause; .key works before a cause exists.
    • First merge steps: linking to node invocations, connect with direction, export, map/mapWithPattern, and toJSON returning null when unlinked.
    • Deprecated setPreExisting and setDefault; removed toOpaqueRef.
    • Made the recipe name parameter optional.

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.

1 issue found across 1 file

Prompt for AI agents (all 1 issues)

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


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

<violation number="1" location="docs/specs/recipe-construction/rollout-plan.md:22">
Typo: the checklist item says &quot;one ore more&quot; but it should read &quot;one or more&quot; for correct English.</violation>
</file>

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

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
@seefeldb seefeldb merged commit 79648aa into main Oct 24, 2025
1 check passed
@seefeldb seefeldb deleted the spec(runner)/more-detail-in-plan branch October 24, 2025 19:07
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