tewip-ai 0.49.0 → 0.50.0

package/dist/engine/schema/schema-command.js

@@ -13,8 +13,12 @@ import { dirname, resolve } from 'node:path';
 import { changeSummary } from '../quality/grounding.js';
 import { dialectOf, forwardOnly, irreversible, readFramework } from './frameworks.js';
 import { readSql } from './statements.js';
-const MIGRATION = /(^|\/)(migrations?|db\/migrate|prisma\/migrations|alembic|flyway|sql)\//i;
-const SCHEMA_FILE = /\.(sql)$|(^|\/)(migrations?|db\/migrate)\/.*\.(py|rb|js|ts)$/i;
+const MIGRATION = /(^|\/)(migrations?|migration|db\/migrate|prisma\/migrations|alembic|flyway|changelogs?|sql)\//i;
+// A changelog is often not in a directory that says so. OpenMRS keeps 44 Liquibase changelogs in
+// `api/src/main/resources/` named `liquibase-*.xml`, and the path filter saw none of them - so a
+// real healthcare system with 44 migration files was told it had none. The name counts too.
+const NAMED_MIGRATION = /(^|\/)(liquibase|changelog)[\w.-]*\.(xml|ya?ml|json)$/i;
+const SCHEMA_FILE = /\.(sql)$|(^|\/)(migrations?|migration|db\/migrate)\/.*\.(py|rb|js|ts|php|cs)$/i;
 /** The schema files in this change. */
 export function schemaFiles(cmd) {
     const named = cmd.files !== undefined && cmd.files.length > 0;
@@ -24,7 +28,7 @@ export function schemaFiles(cmd) {
         // Asking for a file by name overrides the path filter: the filter exists to find migrations in
         // a diff of a hundred files, not to argue with somebody who pointed at one. A TypeORM migration
         // sitting outside a migrations/ directory was being answered with "no migration in this change".
-        .filter((f) => (named || SCHEMA_FILE.test(f) || MIGRATION.test(f)) && existsSync(resolve(cmd.root, f)));
+        .filter((f) => (named || SCHEMA_FILE.test(f) || MIGRATION.test(f) || NAMED_MIGRATION.test(f)) && existsSync(resolve(cmd.root, f)));
 }
 /**
  * Tables this migration creates itself.

package/dist/engine/start/start-command.js

@@ -37,6 +37,7 @@ export function lookAround(root, limit = 4000) {
         hasWorkflows: existsSync(join(root, '.github', 'workflows')),
         hasStandards: existsSync(join(root, '.tewip', 'standards')),
         filesSeen: false,
+        stoppedEarly: false,
     };
     let seen = 0;
     // A queue rather than recursion: every directory at one depth is read before any at the next.
@@ -53,8 +54,13 @@ export function lookAround(root, limit = 4000) {
             continue;
         }
         for (const entry of entries) {
-            if (seen >= limit)
+            // Out of budget with entries still unread in *this* directory. Checking only the queue was
+            // not enough: a project whose files sit in one flat directory empties the queue before the
+            // budget runs out, so the look stopped short and did not know it.
+            if (seen >= limit) {
+                found.stoppedEarly = true;
                 break;
+            }
             if (SKIP.has(entry) || (entry.startsWith('.') && entry !== '.github'))
                 continue;
             const full = join(dir, entry);
@@ -78,7 +84,10 @@ export function lookAround(root, limit = 4000) {
                 found.perfReports.push(rel);
             else if (/(k6|lighthouse|summary|lhr)[\w.-]*\.json$/i.test(entry) && info.size < 8_000_000)
                 found.perfReports.push(rel);
-            const inMigrations = /(^|\/)(migrations?|db\/migrate|changelog)\//i.test(rel);
+            // A changelog is often not in a directory that says so: OpenMRS keeps 44 Liquibase files in
+            // `api/src/main/resources/` named `liquibase-*.xml`. The name counts as well as the path.
+            const inMigrations = /(^|\/)(migrations?|migration|db\/migrate|changelogs?)\//i.test(rel) ||
+                /(^|\/)(liquibase|changelog)[\w.-]*\.(xml|ya?ml|json)$/i.test(rel);
             // Reading a file is only worth it for the two questions a name cannot answer: is this a
             // test, and is this a migration. Both are cheap, and both are often wrong from the path.
             const language = languageOf(entry);
@@ -101,6 +110,8 @@ export function lookAround(root, limit = 4000) {
             }
         }
     }
+    if (seen >= limit && queue.length > 0)
+        found.stoppedEarly = true;
     return found;
 }
 const total = (counts) => [...counts.values()].reduce((a, b) => a + b, 0);
@@ -139,19 +150,19 @@ export function suggest(found) {
     if (tests > 0) {
         out.push({
             command: 'tewip vouch',
-            because: `${tests} test ${tests === 1 ? 'file' : 'files'} here (${named(found.tests)}). This reads them and names the ones that cannot fail - no model, a second or two.`,
+            because: `${found.stoppedEarly ? 'At least ' : ''}${tests} test ${tests === 1 ? 'file' : 'files'} here (${named(found.tests)}). This reads them and names the ones that cannot fail - no model, a second or two.`,
         });
     }
     if (migrations > 0) {
         out.push({
             command: 'tewip schema --diff main...HEAD',
-            because: `${migrations} ${migrations === 1 ? 'migration' : 'migrations'} here (${named(found.migrations)}). This says what a change would do to a live database before it runs.`,
+            because: `${found.stoppedEarly ? 'At least ' : ''}${migrations} ${migrations === 1 ? 'migration' : 'migrations'} here (${named(found.migrations)}). This says what a change would do to a live database before it runs.`,
         });
     }
     if (found.markup > 0) {
         out.push({
             command: 'tewip access',
-            because: `${found.markup} markup ${found.markup === 1 ? 'file' : 'files'} here. This names who cannot use the interface, against the WCAG criterion a regulator would cite.`,
+            because: `${found.stoppedEarly ? 'At least ' : ''}${found.markup} markup ${found.markup === 1 ? 'file' : 'files'} here. This names who cannot use the interface, against the WCAG criterion a regulator would cite.`,
         });
     }
     for (const report of found.perfReports.slice(0, 1)) {
@@ -187,6 +198,11 @@ export function renderStart(found, root) {
     }
     lines.push('## What it will not do here', '');
     const cannot = [];
+    if (found.stoppedEarly) {
+        cannot.push('This project is larger than Tewip looks at from here, so the counts above are **at least** ' +
+            'that many rather than exactly. The commands themselves read everything; only this first ' +
+            'glance is bounded, so that it stays instant.');
+    }
     if (total(found.tests) === 0)
         cannot.push('No test files were found, so nothing can be said about whether your tests could fail or would catch a bug.');
     if (total(found.migrations) === 0)

package/dist/engine/vouch/vouch.js

@@ -216,9 +216,15 @@ const stripLines = (body, lines) => body
     .join(String.fromCharCode(10));
 /** The first line that does something, for the report to quote. */
 function firstRealLine(body) {
-    const line = body
+    const lines = body
         .split(String.fromCharCode(10))
         .map((l) => l.trim())
-        .find((l) => l !== '' && l !== '{' && l !== '}');
-    return (line ?? '').slice(0, 120);
+        .filter((l) => l !== '' && l !== '{' && l !== '}');
+    // A comment is not the evidence. Saleor's tests are written `# given` / `# when / then`, so the
+    // quoted evidence for 166 real findings was the single character "#", which tells a reader
+    // nothing about the test it is accusing. The line that does something is the evidence.
+    // Nor is a line with nothing in it but punctuation: a lone quote closing a docstring, a stray
+    // bracket. Evidence has to be something a reader can recognise as the test doing work.
+    const code = lines.find((l) => !/^(#|\/\/|\/\*|\*|"""|'''|--)/.test(l) && /[A-Za-z0-9]/.test(l));
+    return (code ?? lines[0] ?? '').slice(0, 120);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tewip-ai",
-  "version": "0.49.0",
+  "version": "0.50.0",
   "mcpName": "io.github.hmamut39/tewip",
   "description": "Tewip finds why each failing test failed - real bug, flaky test, environment, test data or changed selector - for any stack, from the command line, CI, or your editor.",
   "keywords": [