Repository navigation
docs: restore the concrete type list in the scaffold AGENTS.md - #1635
Merged
Merged
Conversation
#1632 reduced the "Type everything" section to a pointer. Typing is the one rule check and tsc cannot enforce, so the concrete forms belong where an agent reads first: the row type, action input and result, routing props, the typed Route union and prop<T>(). The rule itself stays in the skill's Default Workflow step 8.
|
Preview for Next: fix the listed keys in the compose file |
vivek7405
marked this pull request as ready for review
October 8, 2026 14:05
3 tasks
vivek7405
pushed a commit
that referenced
this pull request
Oct 8, 2026
Restores every rule #1632 collapsed into a pointer, in AGENTS.md, CONVENTIONS.md, workflow.md, CLAUDE.md, both playbooks and the skill, including the design-token bullet in workflow.md. The saving was about 1,500 tokens per build, small next to a visible drop in how apps look. The partial type-list restore from #1635 is replaced by the original text.
1 task
vivek7405
added a commit
that referenced
this pull request
Oct 8, 2026
Restores every rule #1632 collapsed into a pointer, in AGENTS.md, CONVENTIONS.md, workflow.md, CLAUDE.md, both playbooks and the skill, including the design-token bullet in workflow.md. The saving was about 1,500 tokens per build, small next to a visible drop in how apps look. The partial type-list restore from #1635 is replaced by the original text. Co-authored-by: t <t@t>
2 tasks
1 task
vivek7405
added a commit
that referenced
this pull request
Oct 8, 2026
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.
Closes #1634
Restores a short, concrete list of typing forms under "Type everything" in the scaffold AGENTS.md. #1632 had reduced the section to a pointer, and typing is the one rule
webjs checkandtsccannot enforce, so the forms an agent should reach for now sit in the file it reads first. The rule itself stays canonical in the skill's Default Workflow step 8.What changed
packages/cli/templates/AGENTS.md: four bullets (row type via$inferSelect, action input andActionResult<T>, routing props plusnpx webjsdev types,prop<T>()). About 560 characters, well under the roughly 1k the issue allowed, and it does not restate theunknownrule from step 8.Docs surfaces
website/app/docs/ai-first/page.ts: N/A, it only has a generic one-line "Type everything" tip.Test plan
wsimport error in this environment), nothing new.scaffoldAppgenerates the section with the four bullets (checked in the generated AGENTS.md; the CLI entry point itself fails to start in this environment on the samewsimport error as main).