Skip to content

feat(skills): sync deep-modules and tanstack-tools with Jarvis main - #65

Merged
fveracoechea merged 1 commit into
mainfrom
feat/sync-jarvis-skills
Sep 28, 2026
Merged

fveracoechea merged 1 commit into
mainfrom
feat/sync-jarvis-skills

Conversation

@fveracoechea

Copy link
Copy Markdown
Owner

Summary

terzo-ai/jarvis#135 merged with changes after #33 copied it. This PR copies deep-modules and tanstack-tools from Jarvis main at bc77f72. The other four skills from #33 already match Jarvis main.

 skills/
 ├── deep-modules/
 │   ├── SKILL.md              # find-the-feature steps now live in the router
+│   ├── entry-file-lint.md    # the no-restricted-imports rule, out of module-shape
+│   ├── server-state.md       # env reads and per-request state, out of module-shape
+│   ├── sub-modules.md        # one object per entry file, one level down
 │   ├── finding-features.md
 │   ├── interface-design.md
 │   ├── module-shape.md
 │   └── module-tests.md
 └── tanstack-tools/
     ├── query-mutations.md          # a feature's components import their slice by relative path
     ├── router-creating-a-route.md  # points to deep-modules for non-layout code
     └── start-server-functions.md   # the `$` aggregator is for loaders and other modules

skills/README.md takes the new deep-modules description, and a patch changeset covers the change for consumers.

Evidence

  • Before: diff -r between Jarvis main and Operator main reports 5 changed and 3 missing files in deep-modules, and 3 changed files in tanstack-tools.
    After: diff -r reports no differences for both skills.
  • bun run lint:skills passes: every router is under 150 lines, and a link from SKILL.md reaches each new topic file.
  • On macOS, bun run quality fails on this branch in operator-release/main.test.ts, oxlint-policy.test.ts, and several tests that time out after about 620 s. The same tests fail on a clean origin/main, and they pass on 017d26c. CI on main is green, so this is a macOS-local regression on main and not part of this change.

Merge Danger

Door: two-way

The change is markdown only. A revert restores the #33 copy.

Blast Radius: consumer skills

operator setup copies the new topic files into each consumer project. An agent there that follows deep-modules or tanstack-tools gets the revised rules.

🤖 Generated with Claude Code

Jarvis PR #135 merged with changes after the copy in #33. Copy both skills
from Jarvis main at bc77f72, so Operator ships the reviewed version.

deep-modules states the find-the-feature steps in its router, and moves
sub-modules, the entry-file lint rule, and env and per-request state into
their own topics. tanstack-tools has a feature's components import their
query slice by a relative path, and keeps the `$` aggregator for route
loaders and other modules.
@fveracoechea
fveracoechea merged commit 65e608a into main Sep 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant