lintcn 0.6.0 → 0.7.1

package/dist/discover.d.ts

@@ -1,10 +1,17 @@
 export interface RuleMetadata {
     /** kebab-case rule name from // lintcn:name or derived from folder name */
     name: string;
+    /** runtime rule name parsed from Go `rule.Rule{Name: "..."}`. This is
+     *  the name tsgolint uses in diagnostics and must match --warn flags.
+     *  Falls back to `name` if the Go Name field can't be parsed. */
+    goRuleName: string;
     /** one-line description from // lintcn:description */
     description: string;
     /** original source URL from // lintcn:source */
     source: string;
+    /** severity from // lintcn:severity — 'error' (default) or 'warn'.
+     *  Warnings are displayed with yellow styling and don't cause exit code 1. */
+    severity: 'error' | 'warn';
     /** exported Go variable name like NoFloatingPromisesRule */
     varName: string;
     /** Go package name (= subfolder name, e.g. no_floating_promises) */
@@ -12,5 +19,8 @@ export interface RuleMetadata {
 }
 export declare function parseMetadata(content: string): Record<string, string>;
 export declare function parseRuleVar(content: string): string | undefined;
+/** Extract the Name field from a specific rule.Rule variable's struct literal.
+ *  Scoped to varName to avoid matching Name fields in unrelated structs. */
+export declare function parseGoRuleName(content: string, varName: string): string | undefined;
 export declare function discoverRules(lintcnDir: string): RuleMetadata[];
 //# sourceMappingURL=discover.d.ts.map

package/dist/discover.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"discover.d.ts","sourceRoot":"","sources":["../src/discover.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,YAAY;IAC3B,2EAA2E;IAC3E,IAAI,EAAE,MAAM,CAAA;IACZ,sDAAsD;IACtD,WAAW,EAAE,MAAM,CAAA;IACnB,gDAAgD;IAChD,MAAM,EAAE,MAAM,CAAA;IACd,4DAA4D;IAC5D,OAAO,EAAE,MAAM,CAAA;IACf,oEAAoE;IACpE,WAAW,EAAE,MAAM,CAAA;CACpB;AAOD,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAMrE;AAED,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAGhE;AAED,wBAAgB,aAAa,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,EAAE,CAgD/D"}
+{"version":3,"file":"discover.d.ts","sourceRoot":"","sources":["../src/discover.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,YAAY;IAC3B,2EAA2E;IAC3E,IAAI,EAAE,MAAM,CAAA;IACZ;;qEAEiE;IACjE,UAAU,EAAE,MAAM,CAAA;IAClB,sDAAsD;IACtD,WAAW,EAAE,MAAM,CAAA;IACnB,gDAAgD;IAChD,MAAM,EAAE,MAAM,CAAA;IACd;kFAC8E;IAC9E,QAAQ,EAAE,OAAO,GAAG,MAAM,CAAA;IAC1B,4DAA4D;IAC5D,OAAO,EAAE,MAAM,CAAA;IACf,oEAAoE;IACpE,WAAW,EAAE,MAAM,CAAA;CACpB;AAaD,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAMrE;AAED,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAGhE;AAED;4EAC4E;AAC5E,wBAAgB,eAAe,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAGpF;AAED,wBAAgB,aAAa,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,EAAE,CAsD/D"}

package/dist/discover.js

@@ -7,6 +7,12 @@ import path from 'node:path';
 // and optional import alias (e.g. `r.Rule{` if imported as `r "...rule"`)
 const RULE_VAR_RE = /^\s*var\s+(\w+)\s*=\s*\w*\.?Rule\s*\{/m;
 const METADATA_RE = /^\/\/\s*lintcn:(\w+)\s+(.+)$/gm;
+// buildGoRuleNameRe creates a regex scoped to a specific rule variable's
+// struct literal, e.g. `var FooRule = rule.Rule{ ... Name: "foo" ... }`.
+// This avoids matching a Name field in an unrelated struct earlier in the file.
+function buildGoRuleNameRe(varName) {
+    return new RegExp(`var\\s+${varName}\\s*=\\s*\\w*\\.?Rule\\s*\\{[\\s\\S]*?Name:\\s*"([^"]+)"`);
+}
 export function parseMetadata(content) {
     const meta = {};
     for (const match of content.matchAll(METADATA_RE)) {
@@ -18,6 +24,12 @@ export function parseRuleVar(content) {
     const match = content.match(RULE_VAR_RE);
     return match?.[1];
 }
+/** Extract the Name field from a specific rule.Rule variable's struct literal.
+ *  Scoped to varName to avoid matching Name fields in unrelated structs. */
+export function parseGoRuleName(content, varName) {
+    const match = content.match(buildGoRuleNameRe(varName));
+    return match?.[1];
+}
 export function discoverRules(lintcnDir) {
     if (!fs.existsSync(lintcnDir)) {
         return [];
@@ -47,10 +59,15 @@ export function discoverRules(lintcnDir) {
             if (!varName)
                 continue;
             const meta = parseMetadata(content);
+            const severity = meta.severity === 'warn' ? 'warn' : 'error';
+            const displayName = meta.name || entry.name.replace(/_/g, '-');
+            const goRuleName = parseGoRuleName(content, varName) || displayName;
             rules.push({
-                name: meta.name || entry.name.replace(/_/g, '-'),
-                description: meta.description || '',
-                source: meta.source || '',
+                name: displayName,
+                goRuleName,
+                description: meta.description,
+                source: meta.source,
+                severity,
                 varName,
                 packageName: entry.name,
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lintcn",
-  "version": "0.6.0",
+  "version": "0.7.1",
   "type": "module",
   "description": "The shadcn for type-aware TypeScript lint rules. Browse, pick, and copy rules into your project.",
   "bin": "dist/cli.js",
@@ -60,6 +60,7 @@
     "tar": "^7.5.12"
   },
   "scripts": {
-    "build": "rm -rf dist *.tsbuildinfo && tsc && chmod +x dist/cli.js"
+    "build": "rm -rf dist *.tsbuildinfo && tsc && chmod +x dist/cli.js",
+    "cli": "tsx src/cli"
   }
 }

package/src/cache.ts

@@ -16,9 +16,9 @@ import { extract } from 'tar'
 import { execAsync } from './exec.ts'
 
 // Pinned tsgolint fork commit — updated with each lintcn release.
-// Uses remorses/tsgolint fork which adds internal/runner.Run().
-// Only 1 commit on top of upstream — zero modifications to existing files.
-export const DEFAULT_TSGOLINT_VERSION = 'e945641eabec22993eda3e7c101692e80417e0ea'
+// Uses remorses/tsgolint fork which adds internal/runner.Run() and
+// TSGOLINT_SNAPSHOT_CWD env var for cwd-relative test snapshots.
+export const DEFAULT_TSGOLINT_VERSION = '518fa0d395effb07a45070643a0cb1a9cf202ce9'
 
 // Pinned typescript-go base commit from microsoft/typescript-go (before patches).
 // Patches from tsgolint/patches/ are applied on top during setup.

package/src/cli.ts

@@ -11,6 +11,7 @@ import { listRules } from './commands/list.ts'
 import { removeRule } from './commands/remove.ts'
 import { clean } from './commands/clean.ts'
 import { DEFAULT_TSGOLINT_VERSION } from './cache.ts'
+import { findLintcnDir } from './paths.ts'
 
 const require = createRequire(import.meta.url)
 const packageJson = require('../package.json') as { version: string }
@@ -18,11 +19,13 @@ const packageJson = require('../package.json') as { version: string }
 const cli = goke('lintcn')
 
 cli
-  .command('add <url>', 'Add a rule by GitHub URL. Fetches the whole folder into .lintcn/{rule}/')
-  .example('# Add a rule folder')
+  .command('add <url>', 'Add rules by GitHub URL. Supports single rule folders, .lintcn/ directories, or full repo URLs.')
+  .example('# Add a single rule folder')
   .example('lintcn add https://github.com/oxc-project/tsgolint/tree/main/internal/rules/no_floating_promises')
   .example('# Add by file URL (auto-fetches the whole folder)')
   .example('lintcn add https://github.com/oxc-project/tsgolint/blob/main/internal/rules/await_thenable/await_thenable.go')
+  .example('# Add all rules from a repo (downloads .lintcn/ folder)')
+  .example('lintcn add https://github.com/someone/their-project')
   .action(async (url) => {
     await addRule(url)
   })
@@ -43,12 +46,17 @@ cli
 cli
   .command('lint', 'Build custom tsgolint binary and run it against the project')
   .option('--rebuild', 'Force rebuild even if cached binary exists')
+  .option('--fix', 'Automatically fix violations')
   .option('--tsconfig <path>', 'Path to tsconfig.json')
   .option('--list-files', 'List matched files')
+  .option('--all-warnings', 'Show warnings for all files, not just git-changed ones')
   .option('--tsgolint-version [version]', 'Override the pinned tsgolint version (tag or commit). For testing unreleased tsgolint versions.')
   .action(async (options) => {
     const tsgolintVersion = (options.tsgolintVersion as string) || DEFAULT_TSGOLINT_VERSION
     const passthroughArgs: string[] = []
+    if (options.fix) {
+      passthroughArgs.push('--fix')
+    }
     if (options.tsconfig) {
       passthroughArgs.push('--tsconfig', options.tsconfig as string)
     }
@@ -64,6 +72,7 @@ cli
       rebuild: !!options.rebuild,
       tsgolintVersion,
       passthroughArgs,
+      allWarnings: !!options.allWarnings,
     })
     process.exit(exitCode)
   })
@@ -73,6 +82,10 @@ cli
   .option('--rebuild', 'Force rebuild even if cached binary exists')
   .option('--tsgolint-version [version]', 'Override the pinned tsgolint version (tag or commit). For testing unreleased tsgolint versions.')
   .action(async (options) => {
+    if (!findLintcnDir()) {
+      console.log('No .lintcn/ directory found. Run `lintcn add <url>` to add rules.')
+      return
+    }
     const tsgolintVersion = (options.tsgolintVersion as string) || DEFAULT_TSGOLINT_VERSION
     const binaryPath = await buildBinary({ rebuild: !!options.rebuild, tsgolintVersion })
     console.log(binaryPath)

package/src/commands/add.ts

@@ -13,36 +13,57 @@ import { ensureTsgolintSource, DEFAULT_TSGOLINT_VERSION } from '../cache.ts'
 interface ParsedGitHubUrl {
   owner: string
   repo: string
-  ref: string
-  /** Path to the directory containing the rule files */
+  /** Branch/tag/commit. Undefined for bare repo URLs — resolve via API. */
+  ref: string | undefined
+  /** Path to the directory containing the rule files. Empty string for repo root. */
   dirPath: string
   /** Set when URL points to a specific file (not a folder) */
   fileName?: string
 }
 
-/** Parse GitHub blob/tree/raw URLs into components.
- *  Ref is assumed to be the first path component after blob/tree —
- *  branch names with slashes (e.g. feature/foo) are not supported. */
+/** Parse any GitHub URL into components.
+ *  Supports: bare repo, /tree/ folders, /blob/ files, raw.githubusercontent.com.
+ *  Ref is the first path component after blob/tree — branch names with slashes
+ *  (e.g. feature/foo) are not supported. */
 function parseGitHubUrl(url: string): ParsedGitHubUrl | null {
-  // GitHub blob URLs: github.com/owner/repo/blob/ref/path/to/file.go
-  let match = url.match(/^https?:\/\/github\.com\/([^/]+)\/([^/]+)\/blob\/([^/]+)\/(.+)$/)
-  if (match) {
-    const [, owner, repo, ref, filePath] = match
+  let hostname: string
+  let segments: string[]
+  try {
+    const u = new URL(url)
+    hostname = u.hostname
+    segments = u.pathname.replace(/\/$/, '').split('/').filter(Boolean)
+  } catch {
+    return null
+  }
+
+  // raw.githubusercontent.com/owner/repo/ref/path/to/file
+  if (hostname === 'raw.githubusercontent.com') {
+    if (segments.length < 4) return null
+    const [owner, repo, ref, ...rest] = segments
+    const filePath = rest.join('/')
     return { owner, repo, ref, dirPath: path.posix.dirname(filePath), fileName: path.posix.basename(filePath) }
   }
 
-  // GitHub tree URLs: github.com/owner/repo/tree/ref/path/to/folder
-  match = url.match(/^https?:\/\/github\.com\/([^/]+)\/([^/]+)\/tree\/([^/]+)\/(.+)$/)
-  if (match) {
-    const [, owner, repo, ref, dirPath] = match
-    return { owner, repo, ref, dirPath }
+  if (hostname !== 'github.com') return null
+  if (segments.length < 2) return null
+
+  const [owner, repo, kind, ref, ...rest] = segments
+
+  // Bare repo URL: github.com/owner/repo
+  if (!kind) {
+    return { owner, repo, ref: undefined, dirPath: '' }
+  }
+
+  const subPath = rest.join('/')
+
+  if (kind === 'tree') {
+    if (!ref || !subPath) return null
+    return { owner, repo, ref, dirPath: subPath }
   }
 
-  // raw.githubusercontent.com URLs
-  match = url.match(/^https?:\/\/raw\.githubusercontent\.com\/([^/]+)\/([^/]+)\/([^/]+)\/(.+)$/)
-  if (match) {
-    const [, owner, repo, ref, filePath] = match
-    return { owner, repo, ref, dirPath: path.posix.dirname(filePath), fileName: path.posix.basename(filePath) }
+  if (kind === 'blob') {
+    if (!ref || !subPath) return null
+    return { owner, repo, ref, dirPath: path.posix.dirname(subPath), fileName: path.posix.basename(subPath) }
   }
 
   return null
@@ -67,6 +88,30 @@ function getGitHubToken(): string | undefined {
   }
 }
 
+/** Resolve the default branch for a repo (e.g. "main", "master"). */
+async function resolveDefaultBranch(owner: string, repo: string): Promise<string> {
+  const apiUrl = `https://api.github.com/repos/${owner}/${repo}`
+  const headers: Record<string, string> = {
+    'Accept': 'application/vnd.github.v3+json',
+    'User-Agent': 'lintcn',
+  }
+  const token = getGitHubToken()
+  if (token) {
+    headers['Authorization'] = `Bearer ${token}`
+  }
+  const response = await fetch(apiUrl, { headers })
+  if (!response.ok) {
+    throw new Error(`GitHub API error: ${response.status} ${response.statusText}\n  ${apiUrl}`)
+  }
+  const data = (await response.json()) as { default_branch: string }
+  return data.default_branch
+}
+
+/** Files/dirs in .lintcn/ that are generated and should not be treated as rule folders. */
+const LINTCN_GENERATED = new Set([
+  '.tsgolint', '.gitignore', 'go.work', 'go.work.sum', 'go.mod', 'go.sum',
+])
+
 /** List files in a GitHub directory via the Contents API. */
 async function listGitHubFolder(owner: string, repo: string, dirPath: string, ref: string): Promise<GitHubContentItem[]> {
   const apiUrl = `https://api.github.com/repos/${owner}/${repo}/contents/${dirPath}?ref=${ref}`
@@ -118,71 +163,53 @@ function ensureSourceComment(content: string, sourceUrl: string): string {
   return lines.join('\n')
 }
 
-export async function addRule(url: string): Promise<void> {
-  const parsed = parseGitHubUrl(url)
-  if (!parsed) {
-    throw new Error(
-      'Only GitHub URLs are supported. Pass a /blob/ (file) or /tree/ (folder) URL.\n' +
-      'Example: lintcn add https://github.com/oxc-project/tsgolint/tree/main/internal/rules/no_floating_promises',
-    )
-  }
-
-  const { owner, repo, ref, dirPath } = parsed
+/** Download a single rule folder into .lintcn/{folderName}/.
+ *  Overwrites existing folder if present.
+ *  Returns true if the rule was added, false if skipped (no .go files). */
+async function downloadSingleRule(
+  owner: string, repo: string, ref: string, dirPath: string,
+  lintcnDir: string, sourceUrl: string,
+): Promise<boolean> {
   const folderName = path.posix.basename(dirPath)
-
-  console.log(`Fetching ${owner}/${repo}/${dirPath}...`)
   const items = await listGitHubFolder(owner, repo, dirPath, ref)
 
-  // Filter for .go and .json files
   const filesToFetch = items.filter((item) => {
     return item.type === 'file' && item.download_url && (item.name.endsWith('.go') || item.name.endsWith('.json'))
   })
 
   if (filesToFetch.length === 0) {
-    throw new Error(`No .go files found in ${dirPath}. Is this a rule folder?`)
-  }
-
-  // Warn if this doesn't look like a single-rule folder (too many main .go files)
-  const mainGoFiles = filesToFetch.filter((f) => {
-    return f.name.endsWith('.go') && !f.name.endsWith('_test.go') && f.name !== 'options.go'
-  })
-  if (mainGoFiles.length > 3) {
-    console.warn(
-      `Warning: folder has ${mainGoFiles.length} non-test .go files. ` +
-      `This may be a directory of multiple rules — consider using a more specific URL.`,
-    )
+    console.warn(`  Skipping ${folderName}/ — no .go files found`)
+    return false
   }
 
-  const lintcnDir = getLintcnDir()
   const ruleDir = path.join(lintcnDir, folderName)
 
-  // Clean existing rule folder if it exists
   if (fs.existsSync(ruleDir)) {
     fs.rmSync(ruleDir, { recursive: true })
-    console.log(`Overwriting existing ${folderName}/`)
+    console.log(`  Overwriting existing ${folderName}/`)
   }
 
   fs.mkdirSync(ruleDir, { recursive: true })
 
-  // Fetch and write all files
   for (const item of filesToFetch) {
     let content = await fetchFile(item.download_url!)
 
-    // Add lintcn:source comment to the main rule file (same name as folder)
     if (item.name === `${folderName}.go`) {
-      content = ensureSourceComment(content, url)
+      content = ensureSourceComment(content, sourceUrl)
     }
 
     fs.writeFileSync(path.join(ruleDir, item.name), content)
-    console.log(`  ${item.name}`)
+    console.log(`    ${item.name}`)
   }
 
-  console.log(`Added ${folderName}/ (${filesToFetch.length} files)`)
+  console.log(`  Added ${folderName}/ (${filesToFetch.length} files)`)
+  return true
+}
 
-  // Ensure tsgolint source is available
+/** Ensure tsgolint source, refresh symlink, regenerate go.work/go.mod. */
+async function finalizeLintcnDir(lintcnDir: string): Promise<void> {
   const tsgolintDir = await ensureTsgolintSource(DEFAULT_TSGOLINT_VERSION)
 
-  // Create/refresh .tsgolint symlink for gopls
   const tsgolintLink = path.join(lintcnDir, '.tsgolint')
   try {
     fs.lstatSync(tsgolintLink)
@@ -195,3 +222,115 @@ export async function addRule(url: string): Promise<void> {
   generateEditorGoFiles(lintcnDir)
   console.log('Editor support files generated (go.work, go.mod)')
 }
+
+/** Download all rule subfolders from a remote .lintcn/ directory.
+ *  Each subfolder is treated as a separate rule. Local rules not present
+ *  in the remote are preserved (merge, not replace). */
+async function addLintcnFolder(
+  owner: string, repo: string, ref: string, lintcnPath: string, sourceUrl: string,
+): Promise<void> {
+  console.log(`Fetching .lintcn/ from ${owner}/${repo}...`)
+  const items = await listGitHubFolder(owner, repo, lintcnPath, ref)
+
+  const ruleDirs = items.filter((item) => {
+    return item.type === 'dir' && !LINTCN_GENERATED.has(item.name) && !item.name.startsWith('.')
+  })
+
+  if (ruleDirs.length === 0) {
+    throw new Error(`No rule folders found in ${lintcnPath}. Is this a .lintcn/ directory?`)
+  }
+
+  console.log(`Found ${ruleDirs.length} rule(s)`)
+
+  const lintcnDir = getLintcnDir()
+
+  let added = 0
+  for (const dir of ruleDirs) {
+    const ruleDirPath = lintcnPath ? `${lintcnPath}/${dir.name}` : dir.name
+    const ok = await downloadSingleRule(owner, repo, ref, ruleDirPath, lintcnDir, sourceUrl)
+    if (ok) added++
+  }
+
+  if (added === 0) {
+    throw new Error(`No rule folders with .go files found in ${lintcnPath}.`)
+  }
+
+  await finalizeLintcnDir(lintcnDir)
+  console.log(`\nDone — added ${added} rule(s) from ${owner}/${repo}`)
+}
+
+export async function addRule(url: string): Promise<void> {
+  const parsed = parseGitHubUrl(url)
+  if (!parsed) {
+    throw new Error(
+      'Only GitHub URLs are supported.\n' +
+      'Examples:\n' +
+      '  lintcn add https://github.com/someone/their-project\n' +
+      '  lintcn add https://github.com/oxc-project/tsgolint/tree/main/internal/rules/no_floating_promises',
+    )
+  }
+
+  const { owner, repo, fileName } = parsed
+  let { ref, dirPath } = parsed
+
+  // Bare repo URL — resolve default branch and look for .lintcn/ at root
+  if (ref === undefined) {
+    ref = await resolveDefaultBranch(owner, repo)
+    console.log(`Resolved default branch: ${ref}`)
+
+    const rootItems = await listGitHubFolder(owner, repo, '', ref)
+    const lintcnEntry = rootItems.find((item) => item.type === 'dir' && item.name === '.lintcn')
+    if (!lintcnEntry) {
+      throw new Error(
+        `No .lintcn/ directory found in ${owner}/${repo}.\n` +
+        'The repo needs a .lintcn/ folder with rule subfolders.',
+      )
+    }
+
+    await addLintcnFolder(owner, repo, ref, '.lintcn', url)
+    return
+  }
+
+  // For blob/raw URLs, dirPath already points at the parent folder
+  // For tree URLs, dirPath is the folder itself — but it might be a .lintcn/ collection
+  if (!fileName) {
+    const items = await listGitHubFolder(owner, repo, dirPath, ref)
+    const hasGoFiles = items.some((i) => i.type === 'file' && i.name.endsWith('.go'))
+    const hasCandidateSubdirs = items.some((i) => {
+      return i.type === 'dir' && !LINTCN_GENERATED.has(i.name) && !i.name.startsWith('.')
+    })
+    const isLintcnDir = path.posix.basename(dirPath) === '.lintcn'
+
+    // Collection if: explicitly .lintcn/, or has subdirs but no .go files at root.
+    // A single rule folder with testdata/ subdirs won't be misdetected because
+    // it also has .go files at root.
+    if (isLintcnDir || (!hasGoFiles && hasCandidateSubdirs)) {
+      await addLintcnFolder(owner, repo, ref, dirPath, url)
+      return
+    }
+  }
+
+  // Single rule — download one folder
+  const folderName = path.posix.basename(dirPath)
+
+  console.log(`Fetching ${owner}/${repo}/${dirPath}...`)
+  const lintcnDir = getLintcnDir()
+
+  // Warn if this doesn't look like a single-rule folder (too many main .go files)
+  const items = await listGitHubFolder(owner, repo, dirPath, ref)
+  const mainGoFiles = items.filter((f) => {
+    return f.type === 'file' && f.name.endsWith('.go') && !f.name.endsWith('_test.go') && f.name !== 'options.go'
+  })
+  if (mainGoFiles.length > 3) {
+    console.warn(
+      `Warning: folder has ${mainGoFiles.length} non-test .go files. ` +
+      `This may be a directory of multiple rules — consider using a more specific URL.`,
+    )
+  }
+
+  const ok = await downloadSingleRule(owner, repo, ref, dirPath, lintcnDir, url)
+  if (!ok) {
+    throw new Error(`No .go files found in ${dirPath}. Is this a rule folder?`)
+  }
+  await finalizeLintcnDir(lintcnDir)
+}

package/src/commands/lint.ts

@@ -2,9 +2,10 @@
 // Handles Go workspace generation, compilation with caching, and execution.
 
 import fs from 'node:fs'
+import path from 'node:path'
 import { spawn } from 'node:child_process'
-import { requireLintcnDir } from '../paths.ts'
-import { discoverRules } from '../discover.ts'
+import { requireLintcnDir, findLintcnDir } from '../paths.ts'
+import { discoverRules, type RuleMetadata } from '../discover.ts'
 import { generateBuildWorkspace, generateEditorGoFiles } from '../codegen.ts'
 import { ensureTsgolintSource, validateVersion, cachedBinaryExists, getBinaryPath, getBuildDir, getBinDir } from '../cache.ts'
 import { computeContentHash } from '../hash.ts'
@@ -99,16 +100,41 @@ export async function lint({
   rebuild,
   tsgolintVersion,
   passthroughArgs,
+  allWarnings,
 }: {
   rebuild: boolean
   tsgolintVersion: string
   passthroughArgs: string[]
+  allWarnings: boolean
 }): Promise<number> {
+  if (!findLintcnDir()) {
+    console.log('No .lintcn/ directory found. Run `lintcn add <url>` to add rules.')
+    return 0
+  }
+
   const binaryPath = await buildBinary({ rebuild, tsgolintVersion })
 
-  // run the binary with passthrough args, inheriting stdio
+  // Discover rules to inject --warn flags for warning-severity rules.
+  // buildBinary already discovered rules for compilation, but we need the
+  // metadata here to know which rules are warnings at runtime.
+  const lintcnDir = requireLintcnDir()
+  const rules = discoverRules(lintcnDir)
+  const warnArgs = buildWarnArgs(rules)
+
+  // By default, limit warnings to git-changed files so they don't flood
+  // the output in large codebases. --all-warnings bypasses this filter.
+  const hasWarnRules = rules.some((r) => r.severity === 'warn')
+  let warnFileArgs: string[] = []
+  if (hasWarnRules && allWarnings) {
+    warnFileArgs = ['--all-warnings']
+  } else if (hasWarnRules) {
+    warnFileArgs = await buildWarnFileArgs()
+  }
+
+  // run the binary with --warn + --warn-file/--all-warnings flags + passthrough args
+  const allArgs = [...warnArgs, ...warnFileArgs, ...passthroughArgs]
   return new Promise((resolve) => {
-    const proc = spawn(binaryPath, passthroughArgs, {
+    const proc = spawn(binaryPath, allArgs, {
       stdio: 'inherit',
     })
 
@@ -122,3 +148,59 @@ export async function lint({
     })
   })
 }
+
+/** Build --warn flags for rules with severity 'warn'.
+ *  Uses goRuleName (parsed from Go source) to match the runtime name
+ *  that tsgolint uses in diagnostics, avoiding silent mismatches. */
+function buildWarnArgs(rules: RuleMetadata[]): string[] {
+  const args: string[] = []
+  for (const rule of rules) {
+    if (rule.severity === 'warn') {
+      args.push('--warn', rule.goRuleName)
+    }
+  }
+  return args
+}
+
+/** Get git-changed files and build --warn-file flags so warnings only
+ *  appear for new/modified code. Returns [] if git is unavailable or not
+ *  a git repo — the runner will then show no warnings (safe default).
+ *  Linting must never crash from this. */
+async function buildWarnFileArgs(): Promise<string[]> {
+  try {
+    // Get git repo root to resolve relative paths to absolute.
+    const topLevelResult = await execAsync('git', ['rev-parse', '--show-toplevel'], { stdio: 'pipe' }).catch(() => null)
+    if (!topLevelResult) return []
+    const repoRoot = topLevelResult.stdout.trim()
+
+    // Changed files (staged + unstaged vs HEAD)
+    const diffResult = await execAsync('git', ['diff', '--name-only', 'HEAD'], { stdio: 'pipe' }).catch(() => null)
+    // Untracked files (new files not yet committed)
+    const untrackedResult = await execAsync('git', ['ls-files', '--others', '--exclude-standard'], { stdio: 'pipe' }).catch(() => null)
+
+    const files = new Set<string>()
+
+    for (const result of [diffResult, untrackedResult]) {
+      if (!result) continue
+      for (const line of result.stdout.split('\n')) {
+        const trimmed = line.trim()
+        if (trimmed) {
+          // Resolve to absolute path so it matches SourceFile.FileName() in the runner.
+          files.add(path.resolve(repoRoot, trimmed))
+        }
+      }
+    }
+
+    // No changed files → no --warn-file flags → runner shows no warnings (clean tree)
+    if (files.size === 0) return []
+
+    const args: string[] = []
+    for (const file of files) {
+      args.push('--warn-file', file)
+    }
+    return args
+  } catch {
+    // git not installed, not a repo, or any other failure — no warnings shown
+    return []
+  }
+}

package/src/commands/list.ts

@@ -20,10 +20,13 @@ export function listRules(): void {
 
   console.log('Installed rules:\n')
 
-  const maxNameLen = Math.max(...rules.map((r) => { return r.name.length }))
+  const maxNameLen = Math.max(...rules.map((r) => {
+    return r.name.length + (r.severity === 'warn' ? 7 : 0) // ' (warn)' = 7 chars
+  }))
 
   for (const rule of rules) {
-    const name = rule.name.padEnd(maxNameLen + 2)
+    const suffix = rule.severity === 'warn' ? ' (warn)' : ''
+    const name = (rule.name + suffix).padEnd(maxNameLen + 2)
     const desc = rule.description || '(no description)'
     console.log(`  ${name}${desc}`)
   }