Skip to content

docs(agents): clarify unresolved viewer ownership #41

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wstein wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

docs(agents): clarify unresolved viewer ownership #41

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 13 additions & 5 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,14 @@ The live analysis effect clears auto-capture progress when its settings change.
[PuzlWerk already exists](https://github.com/wstein/puzlwerk). Its target ownership
is the shared ReScript cube mechanics, state model, coordinates, cubie/sticker
identity, orientation/symmetry semantics, notation/codecs and reusable solver
integration; its viewer boundary uses TypeScript. This does not mean a package
is released or CubeAssembler already consumes it. See
integration. **Shared TypeScript viewer ownership is unresolved:** [Cubistry
#21](https://github.com/wstein/cubistry/issues/21) originally proposed a
PuzlWerk viewer, but the current [PuzlWerk ecosystem boundary](https://github.com/wstein/puzlwerk/blob/main/README.md#ecosystem-boundary)
excludes browser viewers, while [Cubistry #60](https://github.com/wstein/cubistry/issues/60)
assigns the current renderer to Cubistry. Neither is an approved silent
replacement of #21. Do not add or migrate a shared viewer until the owner
resolves this discrepancy and separately approves M2. Repository existence
does not mean a package is released or CubeAssembler consumes it. See
[Cubistry #21](https://github.com/wstein/cubistry/issues/21),
[local tracking #29](https://github.com/wstein/cube-assembler/issues/29),
[PuzlWerk #5](https://github.com/wstein/puzlwerk/issues/5) and
Expand All @@ -79,9 +85,11 @@ Each implementation/migration phase needs explicit human approval.
CubeAssembler remains the independent scanning/assembly application. Preserve
its capture, color review, profiles, assembly, UI, renderer and existing
2×2–7×7 behavior/tests until an approved replacement and migration plan exist.
Necessary fixes remain allowed in their authorized scope. Before expanding
reusable cube or viewer primitives, check PuzlWerk ownership and track the
shared change rather than building a second permanent engine. Do not silently
Necessary fixes remain allowed in their authorized scope. Before expanding reusable cube mechanics, check PuzlWerk ownership and track
the shared change rather than building a second permanent engine. Before
extending reusable viewer primitives, consult the unresolved viewer ownership
gate in Cubistry #21; preserve the working app renderer and avoid a second
permanent rendering engine. Do not silently
move code, add unapproved dependencies or delete working functionality.

Cubacle remains a separate optional Rust solver provider, not Rust code inside
Expand Down
Loading