Repository navigation
chore(ai-tool-config): tighten Claude Code rules on checks, public config and v-html - #109
Open
joris-sevenlab wants to merge 1 commit into
Open
joris-sevenlab wants to merge 1 commit into
joris-sevenlab wants to merge 1 commit into
Conversation
…nfig and v-html - global.md: run `pnpm run typecheck` (wired by essentials as `nuxi typecheck`, and run in CI) next to lint and tests, and never get checks green by suppressing them. - security.md: nothing secret under `runtimeConfig.public`; never render user-controlled content with `v-html` (no sanitizer ships and `xssValidator` is off). ai-tool-config 1.2.1.
joris-sevenlab
marked this pull request as ready for review
October 21, 2026 09:04
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
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.
Folds the few generic, stack-correct lines from a set of retired always-on skills into the scaffolded Claude Code rules. The source is a skills audit: https://claude.ai/artifact/FWspXMUnSyi2rKLsLuojzV
Folded
global.mdpnpm run typecheckto the "before considering work complete" line.essentialswires it asnuxi typecheckand CI runs it.global.md(same line)@ts-ignore,as any,eslint-disable, a skipped or deleted test, orgit commit --no-verify.security.mdruntimeConfig.public, because it is serialized into every page.security.mdv-html. No sanitizer ships andxssValidatoris off.shared:ai-tool-config1.2.0 → 1.2.1 (pnpm bump). Changelog entry added under Unreleased.collectDocsis unchanged, so no writer feature needs a bump.Deliberately not folded
buildscript. It stays out of the always-loaded gate because it is slow, and CI and Docker already run it.nuxi typecheckalready runs vue-tsc.brainstorming/task-arbiterskills already cover it.security.md..string().email(): that is Zod 3. The rules use Zod 4z.email().server/middleware/01.origin-check.tsalready covers it.routeRules:nuxt-securityowns the headers, and that is documented in AGENTS.md, which CLAUDE.md imports.nuxt-securityrateLimiter is on by default.npm audit: CI runs the package manager audit..envhandling: already insecurity.md.runtimeConfig/env mapping is already inglobal.md.buildModules,@nuxtjs/tailwindcss, Bearer tokens in a store).Tests
pnpm exec vitest run packages/preset-nuxt4/test/claude-rules.test.ts: 2/2 pass.pnpm test: 101 files, 1088 passed, 1 skipped. The first, cold run (177 s) reported 3 failures in 1 file. Two reruns were fully green (~10 s), so it looks like a timeout flake unrelated to this change.pnpm tsc: clean.pnpm bump:check origin/main: OK.pnpm changelog:check: no base ref locally; CI will check it on the PR.