docs: viewer plan — Task 0c, Mantine 9.5.2 and the LLM docs path - #107
Merged
Conversation
… actually uses The version is the small half. The task ships the MCP server pinned to the version it documents, a vendored llms.txt for sessions without it, and the instruction that makes an implementer reach for either — a guessed prop compiles, renders, and is still wrong. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe plan adds a Mantine 9.5.2 upgrade task, version-pinned MCP configuration, offline documentation, agent instructions, scoped validation, and scaffold propagation checks. ChangesMantine plan updates
Estimated code review effort: 1 (Trivial) | ~5 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
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
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.
Adds Task 0c to the chat viewer plan: upgrade Mantine 9.4.1 → 9.5.2 and, more to the point, give the agents building this UI a way to look Mantine up instead of recalling it.
The version is the small half. 9.5.2 is a minor already inside both repos'
^9.4.1range, scoped to the eight@mantine/*packages (a barebun updatewould drag vite, storybook, eslint and React along under one task's name) and gated by each repo's normal suite. No visual-regression apparatus for a minor.The docs path is the point. Three artifacts, each covering a different failure:
.mcp.jsonpinning@mantine/mcp-server@9.5.2— it ships in lockstep with Mantine, so pinning it to the installed version is what makes the answers true rather than merely recent. Four tools:list_items,get_item_doc,get_item_props,search_docs.docs/mantine-llms.txtvendored (42 KB index) for sessions without MCP.llms-full.txtstays out at 4.1 MB — Mantine's guide still advertises ~1.8 MB.AGENTS.md, which is what actually changes behaviour. Tooling nobody is told to use gets used by nobody.Ordering: mantine-kit first, because
create-cliscaffolds fromgit ls-filesand copies the template's ownpackage.json/bun.lock/AGENTS.mdinto every generated app — so whatever the template holds is what chat inherits at Task 1. Console follows and rebases onto Task 0b, which touches the samepackage.json.Verification: two review rounds. The first caught that the new files would never reach a scaffold (untracked, and
git commit -amdoesn't add them), that the kit'sbun run testis watch mode and would hang, and that the MCP check can't be run by the agent that writes the config. The second caught that the probe's version check was a comment rather than an assertion, thatCLAUDE.mdis a 15-byte pointer in both repos so the block belongs inAGENTS.mdalone, and that the probe aborts on re-run.Known and deliberately not fixed here: Task 0b's
zero pixel driftgate is vacuous — console's parity capture renders static design mocks and never loads the app, so it cannot fail. That matters for 0b's extraction, not for this bump; flagged for Matt.🤖 Generated with Claude Code
Summary by CodeRabbit