Skip to content

fix(lint): remove forbidden lint scripts, align coverage/test with n-rules canon#11

Merged
vitaliytv merged 1 commit into
mainfrom
claude/bold-faraday-6ff1fe
Jul 23, 2026
Merged

fix(lint): remove forbidden lint scripts, align coverage/test with n-rules canon#11
vitaliytv merged 1 commit into
mainfrom
claude/bold-faraday-6ff1fe

Conversation

@vitaliytv

Copy link
Copy Markdown
Member

Summary

  • Removes scripts.lint / scripts.lint-* from root package.json — the bun.mdc policy-deny gate (bun/package_json) forbids these; lint runs only via npx @7n/rules lint, not package.json scripts.
  • Fixes scripts.coverage to satisfy the test/package_json gate: now runs npx @7n/rules lint test --no-fix (matches the substring required by the coverage/mutation lint-detector concern shipped in @7n/rules@1.45.0, which supersedes the older @7n/test coverage direct-invocation canon still shown in the stale bundled docs).
  • Fixes scripts.test to include --bun (bun run --bun vitest run), so forked vitest pool processes inherit the Bun engine and can resolve Bun-native built-ins like import { SQL } from 'bun'.
  • bun.lock updated via bun i (bumps @7n/rules devDependency to ^1.45.0 to match what was actually resolved; reconciles stale @7n/tauri-components root lockfile entries against current package.json).

Why

These were pre-existing npx @7n/rules lint policy-deny violations in the repo root package.json, surfaced once it entered a lint delta scope — not caused by any unrelated recent change.

Note: a separate, already-tracked vue/package_json violation in owner/package.json (missing vite-plugin-vue-layouts-next / vue-macros) is intentionally left untouched — tracked at mt/lint-fix-vue-packages-c8869493/.

Test plan

  • npx @7n/rules lint --no-fix bun test → exit 0 (both violations gone)
  • npx @7n/rules lint (delta) → only the pre-existing, separately-tracked vue/package_json violation remains

🤖 Generated with Claude Code

… with n-rules canon

- Drop root package.json scripts.lint / lint-* — lint runs only via
  `npx @7n/rules lint`, per bun.mdc policy-deny gate (bun/package_json).
- scripts.coverage now runs `npx @7n/rules lint test --no-fix`, matching
  the test/package_json substring requirement shipped in @7n/rules 1.45.0
  (new coverage/mutation lint-detector concern).
- scripts.test now runs `bun run --bun vitest run` so forked vitest pool
  processes inherit the Bun engine (required for Bun-native built-ins
  like `import { SQL } from 'bun'`).
- bun.lock updated via `bun i` (bumps @7n/rules to ^1.45.0, reconciles
  stale @7n/tauri-components lockfile entries).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vitaliytv
vitaliytv merged commit 7f64c3f into main Jul 23, 2026
2 of 7 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