Danny Kim
ccfb08f4d5
Follow-up to #20, which was merged before this landed. bot-agents has no `package.json` `imports` aliases (`src/` is flat and everything imports relatively), so no code changes. This records the new rule in the docs: - `.agents/memory/imports-aliases-barrels.md`: aliases may target only a direct subfolder of `src/` (its `index.ts` or a `#x/*` pattern); nothing outside `src/`, not `src/` itself, not deeper folders; such code imports relatively. - `.agents/memory/tests-layout-vitest.md`: `tests/` reach the public entry through a relative import, not a `#plugin` alias. - `.agents/skills/migrate-deno-repo/SKILL.md` and `AGENTS.md`: same rule. `vp check`, `vp test`, and `damem doctor` pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Reviewed-on: #21 Reviewed-by: bot <noreply@capsulizers.com> Co-authored-by: Danny Kim <temeddix@gmail.com> Co-committed-by: Danny Kim <temeddix@gmail.com>
47 lines
2.4 KiB
Markdown
47 lines
2.4 KiB
Markdown
---
|
|
description: Lint coverage never drops below what Deno enforced; every Deno rule maps to an Oxlint rule, a plugin rule, the compiler, or our own rule
|
|
---
|
|
|
|
Hard rule (2026-10-12): after the move, a repo has at least the lint coverage
|
|
it had under Deno: built-in rules (the `recommended` and `jsr` tags plus
|
|
opt-ins like `camelcase`, `eqeqeq`, `guard-for-in`, `no-console`,
|
|
`no-eval`, `no-throw-literal`, `no-non-null-assertion`,
|
|
`verbatim-module-syntax`), `@hugoalh` rules, and the custom plugins. Rules may
|
|
be added, never lost. Formatting may change. "Gap" is not an acceptable end
|
|
state: a Deno rule Oxlint lacks is replaced by a plugin rule or our own rule in
|
|
[[project-lint-plugin]].
|
|
|
|
Before removing any line from `configs.project`, find what else covers it.
|
|
|
|
## How the template mapped Deno's 97 rules
|
|
|
|
- **Same rule** in `configs.project` at `error`.
|
|
- **Renamed equivalent:** `no-new-symbol` → `no-new-native-nonconstructor`;
|
|
`no-window` → `no-restricted-globals: ["error", "window"]`;
|
|
`ban-untagged-ignore` → `unicorn/no-abusive-eslint-disable`;
|
|
`ban-unused-ignore`, `ban-unknown-rule-code` →
|
|
`reportUnusedDisableDirectives`; `camelcase` → built-in `id-match`
|
|
([[naming-id-match]]).
|
|
- **Caught by the parser or `typeCheck`:** `no-dupe-args`, `no-octal`,
|
|
`no-await-in-sync-fn`, `no-import-assertions` (TS2880), `no-sloppy-imports`
|
|
(TS2835), `verbatim-module-syntax` (TS1484).
|
|
- **`no-slow-types`:** `isolatedDeclarations`, enforced by `vp pack` (TS9010)
|
|
in packages.
|
|
- **Vacuous:** JSX rules (no `.tsx` exists) and Deno-only rules.
|
|
- **`@hugoalh` rules** map to Oxlint built-ins in `configs.project`.
|
|
- **deno-import-check's CLI cycle check** → `import/no-cycle`.
|
|
|
|
The template's list in `configs.project` is the baseline to copy: `curly`,
|
|
`guard-for-in`, `no-console`, `no-empty`, `no-regex-spaces`,
|
|
`no-restricted-globals`, `no-var`, `prefer-const`, `import/no-cycle`,
|
|
`import/no-relative-parent-imports`, `typescript/adjacent-overload-signatures`,
|
|
`typescript/consistent-type-imports`, `typescript/no-empty-interface`,
|
|
`typescript/no-explicit-any`, `typescript/no-namespace`,
|
|
`typescript/no-non-null-assertion`, the two `project/*` rules,
|
|
`unicorn/no-abusive-eslint-disable`, `id-match`, and
|
|
`typescript/switch-exhaustiveness-check` with
|
|
`considerDefaultExhaustiveForUnions`.
|
|
|
|
Each migration PR states its parity mapping (or links the template's table in
|
|
cunarist/eslint-plugin-explicit-types#1) so the user can check it.
|