monorepo clean

Author: sarahxsanders

.gitignore

@@ -34,4 +34,6 @@ plugins
 .env
 .vercel
 
-e2e-tests/fixtures/.tracking/*
+e2e-tests/fixtures/.tracking/*
+
+CLAUDE.local.md

src/android/android-wizard-agent.ts

@@ -96,8 +96,6 @@ export const ANDROID_AGENT_CONFIG: FrameworkConfig<AndroidContext> = {
   prompts: {
     projectTypeDetection:
       'This is an Android/Kotlin project. Look for build.gradle or build.gradle.kts files, AndroidManifest.xml, and Kotlin source files (.kt) to confirm.',
-    packageInstallation:
-      'Add the PostHog Android SDK dependency to the app-level build.gradle(.kts) file. Use implementation("com.posthog:posthog-android:<VERSION>"). Check the existing dependency format (Groovy vs Kotlin DSL) and match it.',
     getAdditionalContextLines: (context) => {
       const lines = [
         `Framework docs ID: android (use posthog://docs/frameworks/android for documentation)`,

src/angular/angular-wizard-agent.ts

@@ -60,8 +60,6 @@ export const ANGULAR_AGENT_CONFIG: FrameworkConfig<AngularContext> = {
 
       return [
         `Framework docs ID: ${frameworkId} (use posthog://docs/frameworks/${frameworkId} for documentation)`,
-        'Angular uses dependency injection for services. PostHog should be initialized as a service.',
-        'For standalone components, ensure PostHog is properly provided in the application config.',
       ];
     },
   },

src/django/django-wizard-agent.ts

@@ -7,6 +7,7 @@ import { Integration } from '../lib/constants';
 import fg from 'fast-glob';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
+import { PYTHON_DETECTION_IGNORES } from '../lib/glob-patterns';
 import {
   getDjangoVersion,
   getDjangoProjectType,
@@ -48,7 +49,7 @@ export const DJANGO_AGENT_CONFIG: FrameworkConfig<DjangoContext> = {
 
       const managePyMatches = await fg('**/manage.py', {
         cwd: installDir,
-        ignore: ['**/venv/**', '**/.venv/**', '**/env/**', '**/.env/**'],
+        ignore: PYTHON_DETECTION_IGNORES,
       });
 
       if (managePyMatches.length > 0) {
@@ -77,7 +78,7 @@ export const DJANGO_AGENT_CONFIG: FrameworkConfig<DjangoContext> = {
         ['**/requirements*.txt', '**/pyproject.toml', '**/setup.py'],
         {
           cwd: installDir,
-          ignore: ['**/venv/**', '**/.venv/**', '**/env/**', '**/.env/**'],
+          ignore: PYTHON_DETECTION_IGNORES,
         },
       );
 

src/fastapi/fastapi-wizard-agent.ts

@@ -3,12 +3,7 @@ import type { WizardOptions } from '../utils/types';
 import type { FrameworkConfig } from '../lib/framework-config';
 import { PYTHON_PACKAGE_INSTALLATION } from '../lib/framework-config';
 import { detectPythonPackageManagers } from '../lib/package-manager-detection';
-import { enableDebugLogs } from '../utils/debug';
-import { runAgentWizard } from '../lib/agent-runner';
 import { Integration } from '../lib/constants';
-import clack from '../utils/clack';
-import chalk from 'chalk';
-import * as semver from 'semver';
 import {
   getFastAPIVersion,
   getFastAPIProjectType,
@@ -20,12 +15,14 @@ import {
 import fg from 'fast-glob';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
+import {
+  PYTHON_DETECTION_IGNORES,
+  PYTHON_SOURCE_IGNORES,
+} from '../lib/glob-patterns';
 
 /**
  * FastAPI framework configuration for the universal agent runner
  */
-const MINIMUM_FASTAPI_VERSION = '0.100.0';
-
 export const FASTAPI_AGENT_CONFIG: FrameworkConfig = {
   metadata: {
     name: 'FastAPI',
@@ -49,6 +46,7 @@ export const FASTAPI_AGENT_CONFIG: FrameworkConfig = {
       return undefined;
     },
     getVersionBucket: getFastAPIVersionBucket,
+    minimumVersion: '0.100.0',
     getInstalledVersion: getFastAPIVersion,
     detect: async (options) => {
       const { installDir } = options;
@@ -66,7 +64,7 @@ export const FASTAPI_AGENT_CONFIG: FrameworkConfig = {
         ],
         {
           cwd: installDir,
-          ignore: ['**/venv/**', '**/.venv/**', '**/env/**', '**/.env/**'],
+          ignore: PYTHON_DETECTION_IGNORES,
         },
       );
 
@@ -94,13 +92,7 @@ export const FASTAPI_AGENT_CONFIG: FrameworkConfig = {
         ['**/main.py', '**/app.py', '**/application.py', '**/__init__.py'],
         {
           cwd: installDir,
-          ignore: [
-            '**/venv/**',
-            '**/.venv/**',
-            '**/env/**',
-            '**/.env/**',
-            '**/__pycache__/**',
-          ],
+          ignore: PYTHON_SOURCE_IGNORES,
         },
       );
 
@@ -198,35 +190,3 @@ export const FASTAPI_AGENT_CONFIG: FrameworkConfig = {
     ],
   },
 };
-
-/**
- * FastAPI wizard powered by the universal agent runner.
- */
-export async function runFastAPIWizardAgent(
-  options: WizardOptions,
-): Promise<void> {
-  if (options.debug) {
-    enableDebugLogs();
-  }
-
-  // Check FastAPI version - agent wizard requires >= 0.100.0
-  const fastapiVersion = await getFastAPIVersion(options);
-
-  if (fastapiVersion) {
-    const coercedVersion = semver.coerce(fastapiVersion);
-    if (coercedVersion && semver.lt(coercedVersion, MINIMUM_FASTAPI_VERSION)) {
-      const docsUrl =
-        FASTAPI_AGENT_CONFIG.metadata.unsupportedVersionDocsUrl ??
-        FASTAPI_AGENT_CONFIG.metadata.docsUrl;
-
-      clack.log.warn(
-        `Sorry: the wizard can't help you with FastAPI ${fastapiVersion}. Upgrade to FastAPI ${MINIMUM_FASTAPI_VERSION} or later, or check out the manual setup guide.`,
-      );
-      clack.log.info(`Setup FastAPI manually: ${chalk.cyan(docsUrl)}`);
-      clack.outro('PostHog wizard will see you next time!');
-      return;
-    }
-  }
-
-  await runAgentWizard(FASTAPI_AGENT_CONFIG, options);
-}

src/flask/flask-wizard-agent.ts

@@ -7,6 +7,10 @@ import { Integration } from '../lib/constants';
 import fg from 'fast-glob';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
+import {
+  PYTHON_DETECTION_IGNORES,
+  PYTHON_SOURCE_IGNORES,
+} from '../lib/glob-patterns';
 import {
   getFlaskVersion,
   getFlaskProjectType,
@@ -55,7 +59,7 @@ export const FLASK_AGENT_CONFIG: FrameworkConfig<FlaskContext> = {
         ],
         {
           cwd: installDir,
-          ignore: ['**/venv/**', '**/.venv/**', '**/env/**', '**/.env/**'],
+          ignore: PYTHON_DETECTION_IGNORES,
         },
       );
 
@@ -80,13 +84,7 @@ export const FLASK_AGENT_CONFIG: FrameworkConfig<FlaskContext> = {
         ['**/app.py', '**/wsgi.py', '**/application.py', '**/__init__.py'],
         {
           cwd: installDir,
-          ignore: [
-            '**/venv/**',
-            '**/.venv/**',
-            '**/env/**',
-            '**/.env/**',
-            '**/__pycache__/**',
-          ],
+          ignore: PYTHON_SOURCE_IGNORES,
         },
       );
 

src/javascript-node/javascript-node-wizard-agent.ts

@@ -3,6 +3,9 @@ import type { FrameworkConfig } from '../lib/framework-config';
 import { Integration } from '../lib/constants';
 import { tryGetPackageJson } from '../utils/clack-utils';
 import { detectNodePackageManagers } from '../lib/package-manager-detection';
+import { hasPackageInstalled } from '../utils/package-json';
+import { FRAMEWORK_PACKAGES } from '../javascript-web/utils';
+import { hasLockfileOrDeps } from '../utils/js-detection';
 
 type JavaScriptNodeContext = Record<string, unknown>;
 
@@ -23,7 +26,22 @@ export const JAVASCRIPT_NODE_AGENT_CONFIG: FrameworkConfig<JavaScriptNodeContext
       detectPackageManager: detectNodePackageManagers,
       detect: async (options) => {
         const packageJson = await tryGetPackageJson(options);
-        return !!packageJson;
+        if (!packageJson) {
+          return false;
+        }
+
+        // Exclude projects with known framework packages (handled by
+        // their dedicated detectors earlier in the enum)
+        for (const frameworkPkg of FRAMEWORK_PACKAGES) {
+          if (hasPackageInstalled(frameworkPkg, packageJson)) {
+            return false;
+          }
+        }
+
+        // Catch-all for JS projects without browser signals (those
+        // matched javascript_web already). Require a lockfile or real
+        // dependencies so we don't match bare tooling package.json files.
+        return hasLockfileOrDeps(options.installDir, packageJson);
       },
     },
 
@@ -42,8 +60,6 @@ export const JAVASCRIPT_NODE_AGENT_CONFIG: FrameworkConfig<JavaScriptNodeContext
     prompts: {
       projectTypeDetection:
         'This is a server-side Node.js project. Look for package.json and lockfiles to confirm.',
-      packageInstallation:
-        'Use npm, yarn, pnpm, or bun based on the existing lockfile (package-lock.json, yarn.lock, pnpm-lock.yaml, bun.lockb). Install posthog-node as a regular dependency.',
       getAdditionalContextLines: () => [
         `Framework docs ID: javascript_node (use posthog://docs/frameworks/javascript_node for documentation)`,
       ],

src/javascript-web/javascript-web-wizard-agent.ts

@@ -49,30 +49,25 @@ export const JAVASCRIPT_WEB_AGENT_CONFIG: FrameworkConfig<JavaScriptContext> = {
         }
       }
 
-      // Ensure this is actually a JS project, not just a package.json for tooling
-      const { installDir } = options;
+      // Require a positive browser signal — without one, the project is
+      // more likely a Node.js server/CLI/worker and should fall through
+      // to the javascript_node catch-all (posthog-node is the safer
+      // default since posthog-js crashes without window/document).
+      //
+      // Bundlers alone are NOT a reliable browser signal — Vite/esbuild
+      // are commonly used for server-side builds (Cloudflare Workers, SSR,
+      // Vitest, etc.). Instead we check for:
+      //  1. An HTML entry point (fundamental to browser apps)
+      //  2. A "browser" field in package.json (standard npm browser flag)
+      const hasHtmlEntry = [
+        'index.html',
+        'public/index.html',
+        'src/index.html',
+      ].some((f) => fs.existsSync(path.join(options.installDir, f)));
 
-      // Check for a lockfile
-      const hasLockfile = [
-        'package-lock.json',
-        'yarn.lock',
-        'pnpm-lock.yaml',
-        'bun.lockb',
-        'bun.lock',
-      ].some((lockfile) => fs.existsSync(path.join(installDir, lockfile)));
+      const hasBrowserField = 'browser' in packageJson;
 
-      if (hasLockfile) {
-        return true;
-      }
-
-      // Fallback: check if package.json has actual dependencies
-      const hasDeps =
-        (packageJson.dependencies &&
-          Object.keys(packageJson.dependencies).length > 0) ||
-        (packageJson.devDependencies &&
-          Object.keys(packageJson.devDependencies).length > 0);
-
-      return !!hasDeps;
+      return hasHtmlEntry || hasBrowserField;
     },
   },
 

src/laravel/laravel-wizard-agent.ts

@@ -114,8 +114,6 @@ export const LARAVEL_AGENT_CONFIG: FrameworkConfig<LaravelContext> = {
   prompts: {
     projectTypeDetection:
       'This is a PHP/Laravel project. Look for composer.json, artisan CLI, and app/ directory structure to confirm. Check for Laravel-specific packages like laravel/framework.',
-    packageInstallation:
-      'Use Composer to install packages. Run `composer require posthog/posthog-php` without pinning a specific version.',
     getAdditionalContextLines: (context) => {
       const projectTypeName = context.projectType
         ? getLaravelProjectTypeName(context.projectType)
@@ -135,17 +133,6 @@ export const LARAVEL_AGENT_CONFIG: FrameworkConfig<LaravelContext> = {
         lines.push(`Bootstrap file: ${context.bootstrapFile}`);
       }
 
-      // Add Laravel-specific guidance based on version structure
-      if (context.laravelStructure === 'latest') {
-        lines.push(
-          'Note: Laravel 11+ uses simplified bootstrap/app.php for middleware and providers',
-        );
-      } else {
-        lines.push(
-          'Note: Use app/Http/Kernel.php for middleware, app/Providers for service providers',
-        );
-      }
-
       return lines;
     },
   },

src/lib/__tests__/wizard-tools.test.ts

@@ -47,6 +47,27 @@ describe('resolveEnvPath', () => {
     // edge case: filePath resolves to exactly workingDirectory
     expect(() => resolveEnvPath('/project', '.')).not.toThrow();
   });
+
+  it('strips redundant subdirectory prefix from relative path', () => {
+    // Agent passes "services/mcp/.env" when workingDir is already "/ws/services/mcp"
+    const result = resolveEnvPath('/ws/services/mcp', 'services/mcp/.env');
+    expect(result).toBe(path.resolve('/ws/services/mcp', '.env'));
+  });
+
+  it('strips single-level redundant prefix', () => {
+    const result = resolveEnvPath('/ws/frontend', 'frontend/.env.local');
+    expect(result).toBe(path.resolve('/ws/frontend', '.env.local'));
+  });
+
+  it('does not strip when there is no redundant prefix', () => {
+    const result = resolveEnvPath('/ws/services/mcp', '.env');
+    expect(result).toBe(path.resolve('/ws/services/mcp', '.env'));
+  });
+
+  it('does not strip legitimate nested paths', () => {
+    const result = resolveEnvPath('/ws/services/mcp', 'config/.env');
+    expect(result).toBe(path.resolve('/ws/services/mcp', 'config/.env'));
+  });
 });
 
 // ---------------------------------------------------------------------------