Skip to content
This repository was archived by the owner on Aug 7, 2026. It is now read-only.

chore(rfl-128): add contracts/ mini-workspace (prep for move) - #11

Closed
kpernyer wants to merge 1 commit into
mainfrom
e11/rfl-128-contracts-prep
Closed

kpernyer wants to merge 1 commit into
mainfrom
e11/rfl-128-contracts-prep

Conversation

@kpernyer

@kpernyer kpernyer commented Jul 5, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Adds contracts/Cargo.toml — a minimal standalone workspace (one member: helm-module-contracts)
  • Adds contracts/crates/helm-module-contracts/ — same source and manifest as crates/helm-module-contracts/

Why this is a separate PR

The main move PR (#10) removes crates/helm-module-contracts/ and updates all internal path deps. But the helms CI loads atelier-showcase main via the organism-domain workspace patch, and atelier-showcase main still references the old crates/ location. This creates a deadlock:

This prep PR breaks the deadlock:

  1. Merge this PR → helms main has contracts/ available (old crates/ path still intact)
  2. Atelier/arena-tests PR CIs can now resolve the new path → merge those PRs
  3. Helms PR feat(helm-module-contracts): extract HelmModule + ModuleState from runway-app-host (RFL-128) #10 CI can now load atelier's updated main → passes → merge

Merge order: this prep PR → atelier-showcase #9, arena-tests #7 → helms #10 → runtime-runway #15 → reflective #16

Fixes: https://linear.app/reflective/issue/RFL-128


Note

Low Risk
Additive layout and duplicated crate source only; no dependency rewires or removals in this PR.

Overview
Adds a standalone contracts/ Cargo workspace so helm-module-contracts can be path-dep’d from other repos (e.g. runtime-runway) without pulling in the full helms workspace.

The workspace root is contracts/Cargo.toml with a single member, contracts/crates/helm-module-contracts/, mirroring the existing crate (readiness types, HelmModule / ModuleState mounting contract, and unit tests). The old crates/helm-module-contracts location is unchanged in this PR—this is prep for a later move and cross-repo path updates (RFL-128).

Reviewed by Cursor Bugbot for commit 6907dde. Bugbot is set up for automated code reviews on this repo. Configure here.

Adds contracts/Cargo.toml (a standalone mini-workspace) and
contracts/crates/helm-module-contracts/ so that repos which
will path-dep to the new location can resolve the crate on CI
before the main helm-module-contracts move PR (#10) lands.

The old crates/helm-module-contracts/ location remains intact in this
commit; PR #10 removes it once all downstream path deps are updated.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cursor

cursor Bot commented Jul 5, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_83ca55db-5fa5-4bc9-a708-aa422f182062)

@kpernyer

kpernyer commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by #10 (same contracts/ move, #10 included the full fix and merged).

@kpernyer kpernyer closed this Jul 6, 2026
@kpernyer
kpernyer deleted the e11/rfl-128-contracts-prep branch July 7, 2026 12:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant