Repository navigation
Conversation
Lancetnik
force-pushed
the
feat/card-author
branch
from
October 6, 2026 05:25
736622c to
4d236f4
Compare
Branch coverage
42 files skipped due to complete coverage. |
Lancetnik
added this pull request to stack #131
October 6, 2026 16:11
Lancetnik
marked this pull request as ready for review
October 10, 2026 16:01
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why are these changes needed?
An unknown Card can now be authored, drawn and revised in the current Chat without first installing a Card file. The default-on bundled
card-authorSkill is independent ofrich-views; it exposes the existing vocabulary, worked examples and structured draft operations on demand.Stacked on #129. The PR base is
feat/capabilities-refreshatac7c4d7; the diff contains only the Card author work. Retarget tomainafter #129 merges.Related issue number
Closes #88.
Checks
ruff check .,ruff format --check ., andpytest -m "not integration"locally and they pass.web/, I rebuilt and committed the SPA bundle (npm --prefix web run build).Validation: full Python suite 2345 passed, 4 skipped; feature acceptance/validator suite 37 passed, including cancellation/compaction recovery. Card author and owned-store coverage passed on Python 3.12 and 3.13 as well (43 tests per version).
mypy, strict SPA checks, 543 SPA tests including OpenAPI route/schema contracts, and the source build passed. Updated the AST knowledge graph.Live ordinary Chrome pass on an isolated Profile: two drafts, revisions of each, current-only Save controls, free Save, Replace, Save as copy, Chat restoration, server restart/replay, and a registered interaction retaining edited data after reload. App console/network errors: none.
Independent Standards and Spec reviews completed; all findings were corrected and the final reports have no outstanding findings.
AI assistance
Implemented and reviewed with Codex. This is a draft PR for human ownership review.