Port stranded workspace distillations into the skill tree - #104
Open
christse wants to merge 1 commit into
Open
Conversation
Two distillation passes (50 learnings total) were applied to a downstream workspace's copy of this skill tree; the copy was later dropped in favor of the plugin, stranding the edits in that workspace's git history. This ports them onto current main via 3-way merge. Highlights: - boxel-card-interaction: new skill — empirically-measured click contract (viewCard vs cardComponentModifier vs anchor; Interact/Host click inversion; child-owned pointer events; @mode hydration), verified in a live interaction-lab harness. Registered in index + glossary; corrects the stale glossary claim that viewCard lives on @context. - query-systems: raw-HTTP search uses the new item.on wire grammar (legacy shapes 400); realm scope + page bounds are mandatory (empty realms [] fans out to a federated query). - indexing-operations: /_atomic batch trap, post-push verification gate, file-touch _touched residue + deprecation, CS-12172 stale-computed recovery (unchanged file write, never touch). - publishing-host-mode, css-escape-hatches, parser-output-ground-truth, live-activity-feed-card: new references/patterns. - memory-leak: new completion-gate skill for media/animation-heavy builds. - Assorted refinements across file-def, ui-guidelines, patterns, lint workflow, prefers-wide-format (static-only gotcha), glossary. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Workspace Sync Dry-Run CompleteDry-run completed successfully - no changes were made Dry-Run Results |
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.
Summary
Two distillation passes (consolidating ~50 session learnings) were applied to a downstream workspace's copy of this skill tree. That copy was later dropped in favor of consuming the boxel-cli plugin — the right call — but it stranded the distilled edits in the workspace's git history. This PR ports them onto current
mainvia 3-way merge (base = pre-distill copy, so upstream's own evolution is preserved).New skills
boxel-card-interaction/— empirically-measured click contract for cards that render other cards:viewCardvscardComponentModifiervs anchor, the Interact/Host click-behavior inversion, child-owned pointer events,@modehydration. Verified in a live interaction-lab harness; supersedes the scattered click advice. Also corrects the stale glossary claim thatviewCardlives on@context(it arrives as a component arg).memory-leak/— near-completion lifecycle/allocation audit gate for media/animation-heavy builds.New references & patterns
boxel-environment/references/publishing-host-mode.mdboxel-ui-guidelines/references/css-escape-hatches.md(!important/:deep()/:global()are smells — decision ladder)boxel-file-def/references/parser-output-ground-truth.md(measured parser output per file type)boxel-patterns/patterns/live-activity-feed-card/README.mdNotable refinements
item.onwire grammar (legacy shapes 400, with the silenteq:{"item.on":…}zero-rows trap); realm scope +pagebounds are mandatory ([]fans out to a federated query)./_atomicbatch trap, post-push verification gate,file touch_touched-residue deprecation, CS-12172 stale-computed recovery.staticor it is silently ignored.Notes for reviewers
boxel-platform-bug-reportsskill, bug-report file cross-references) was kept out or reworded; realm-URL examples follow existing precedent in this repo.Skill/(in-app assistant cards) has not been updated — the two trees will drift on these topics until that port happens.🤖 Generated with Claude Code