Fixing .docgap.yaml configuration file name and adding docgap executable to top level folder.

Author: Antigravity

apps/cli/src/index.ts

@@ -8,7 +8,7 @@ const cli = cac('docgap');
 
 cli
     .command('check [cwd]', 'Check documentation drift in the specified directory')
-    .option('--config <path>', 'Path to config file (default: docgap.config.json)')
+    .option('--config <path>', 'Path to config file (default: .docgap.yaml)')
     .option('--strict', 'Exit with code 1 if any drift is detected (useful for CI/CD)')
     .option('--coverage', 'Enable coverage reporting for documented entities')
     .example('  docgap check')
@@ -23,7 +23,7 @@ cli
 // Default command alias to check for convenience (optional, but good DX)
 cli
     .command('[cwd]', 'Check documentation drift (default)')
-    .option('--config <path>', 'Path to config file (default: docgap.config.json)')
+    .option('--config <path>', 'Path to config file (default: .docgap.yaml)')
     .option('--strict', 'Exit with code 1 if any drift is detected (useful for CI/CD)')
     .option('--coverage', 'Enable coverage reporting for documented entities')
     .action(async (cwd, options) => {
@@ -58,7 +58,17 @@ export function bootstrap() {
 // Only run if called directly
 import { fileURLToPath } from 'url';
 // Export for testing
-export const _isMain = (argv1: string, metaUrl: string) => argv1 === fileURLToPath(metaUrl);
+import { realpathSync } from 'fs';
+
+// Export for testing
+export const _isMain = (argv1: string, metaUrl: string) => {
+    const scriptPath = fileURLToPath(metaUrl);
+    try {
+        return argv1 === scriptPath || realpathSync(argv1) === scriptPath;
+    } catch {
+        return argv1 === scriptPath;
+    }
+};
 
 // istanbul ignore next
 /* v8 ignore next 3 */

package.json

@@ -10,6 +10,7 @@
   },
   "packageManager": "[email protected]",
   "devDependencies": {
+    "@docgap/cli": "workspace:*",
     "@vitest/coverage-v8": "^1.1.3",
     "vitest": "^1.1.3"
   }

packages/core/src/index.ts

@@ -49,13 +49,7 @@ async function loadConfig(cwd: string): Promise<DocGapConfig> {
     try {
         return await loadConfigFromPath(configPath);
     } catch (error) {
-        // Fallback to json if needed or throw
-        const jsonPath = path.join(cwd, 'docgap.config.json');
-        try {
-            return await loadConfigFromPath(jsonPath);
-        } catch {
-            throw new Error(`Failed to load config from ${configPath} or ${jsonPath}: ${error}`);
-        }
+        throw new Error(`Failed to load config from ${configPath}: ${error}`);
     }
 }
 

packages/core/tests/unit/index.test.ts

@@ -95,20 +95,7 @@ rules: []
             expect(fg).toHaveBeenLastCalledWith(['s1', 's2'], expect.anything());
         });
 
-        it('falls back to json config if yaml fails', async () => {
-            // yaml fails
-            vi.mocked(fs.readFile).mockRejectedValueOnce(new Error('no yaml'));
-            // json succeeds
-            vi.mocked(fs.readFile).mockResolvedValueOnce(JSON.stringify({
-                rules: []
-            }));
-            vi.mocked(fg).mockResolvedValue([]);
-
-            await runAnalysis('/cwd');
-            expect(fs.readFile).toHaveBeenCalledTimes(2);
-        });
-
-        it('throws if both yaml and json fail', async () => {
+        it('throws if yaml fails', async () => {
             vi.mocked(fs.readFile).mockRejectedValue(new Error('no file'));
 
             await expect(runAnalysis('/cwd')).rejects.toThrow('Failed to load config');