fix: several bugs

Author: transitive-bullshit

docs/pages/project/roadmap.md

@@ -12,6 +12,9 @@
   - social image for docs and launch
     - twitter thread
   - public launch! 🚀
+  - fix confusing rules with empty config
+  - fix cache being in `.gptlint`
+  - fix dry-run should never be cached
 
 ## Post-MVP
 

src/cache.ts

@@ -18,6 +18,7 @@ export class LinterCache<
   TValue extends object = types.LintResult
 > {
   noCache: boolean
+  dryRun: boolean
   cacheDir?: string
   cacheFile?: string
   cache?: Record<string, string>
@@ -27,14 +28,17 @@ export class LinterCache<
   constructor({
     cacheDir,
     cacheFileName = 'cache.json',
-    noCache = false
+    noCache = false,
+    dryRun = false
   }: {
     cacheDir?: string
     cacheFileName?: string
     noCache?: boolean
+    dryRun?: boolean
   }) {
     this.cacheDir = cacheDir
     this.noCache = !!noCache
+    this.dryRun = !!dryRun
     this.cacheFile = this.cacheDir
       ? path.join(this.cacheDir, cacheFileName)
       : undefined
@@ -112,7 +116,7 @@ export class LinterCache<
   }
 
   async flush() {
-    if (!this.cache) return
+    if (!this.cache || this.dryRun) return
 
     const cacheFile = this.cacheFile
     if (cacheFile) {
@@ -132,7 +136,8 @@ export async function createLinterCache<
 >(config: types.ResolvedLinterConfig): Promise<LinterCache<TKey, TValue>> {
   const cache = new LinterCache<TKey, TValue>({
     cacheDir: config.linterOptions.cacheDir,
-    noCache: config.linterOptions.noCache
+    noCache: config.linterOptions.noCache,
+    dryRun: config.linterOptions.dryRun
   })
   await cache.init()
   return cache

src/config.ts

@@ -1,3 +1,5 @@
+import path from 'node:path'
+
 import type { MergeDeep, SetRequired, Simplify } from 'type-fest'
 import type { SimplifyDeep } from 'type-fest/source/merge-deep.js'
 import findCacheDirectory from 'find-cache-dir'
@@ -27,7 +29,7 @@ export type LinterConfigRuleSettings = z.infer<
 >
 
 export const defaultCacheDir =
-  findCacheDirectory({ name: 'gptlint' }) ?? '.gptlint'
+  findCacheDirectory({ name: 'gptlint' }) ?? path.join('.gptlint', 'cache')
 
 export const LLMOptionsSchema = z
   .object({

src/utils.ts

@@ -456,6 +456,9 @@ export function validateLinterInputs({
     console.error(
       `\n${chalk.bold('Error: no rules enabled')} (${chalk.italic('run with --print-config to debug')})\n`
     )
+    console.error(
+      '\nNote: If you want to use the built-in rules with a custom config file, you must enable tem explicitly: https://gptlint.dev/guide/config#config-file\n'
+    )
     gracefulExit(1)
     return false
   }