@delfini/drift-engine 0.2.1 → 0.3.0

package/dist/diff-filter.d.ts

@@ -1,5 +1,5 @@
 /** Why a path or hunk was dropped. */
-export type DropReason = 'lockfile' | 'generated' | 'vendored' | 'fixture' | 'whitespace-only' | 'import-only';
+export type DropReason = 'lockfile' | 'generated' | 'vendored' | 'fixture' | 'whitespace-only' | 'import-only' | 'ignored';
 export interface DroppedPath {
     path: string;
     reason: DropReason;
@@ -17,6 +17,25 @@ export interface FilterDiffResult {
     /** Individual hunks dropped from otherwise-kept files. */
     droppedHunks: DroppedHunk[];
 }
+/** Options controlling which filtering passes `filterDiff` applies. */
+export interface FilterDiffOptions {
+    /**
+     * Apply the built-in noise classifiers (lockfile/generated/vendored/fixture
+     * path-level + whitespace-only/import-only hunk-level). Defaults to `true`
+     * so the legacy single-arg call `filterDiff(diff)` is byte-identical to the
+     * pre-options behaviour (the `enableDiffPreFilter` consumer path). Pass
+     * `false` to run an ignore-only pass that leaves all non-ignored files
+     * verbatim.
+     */
+    builtins?: boolean;
+    /**
+     * User `ignore_code_scope` entries (repo-relative; picomatch@4 dialect via
+     * the shared `isFileInDocScope` predicate). A changed file matching any
+     * entry is dropped whole with reason `'ignored'`, classified BEFORE the
+     * built-ins. Empty / omitted → no ignore dropping (observably no-op).
+     */
+    ignorePaths?: string[];
+}
 /**
  * Filter a unified-diff string deterministically.
  *
@@ -28,6 +47,11 @@ export interface FilterDiffResult {
  * not lose surrounding context.
  *
  * Identical input → identical output (NFR46 reproducibility carries forward).
+ *
+ * `options.builtins` (default `true`) gates the built-in noise classifiers;
+ * `options.ignorePaths` adds user `ignore_code_scope` dropping on top. With
+ * the default options (`{ builtins: true, ignorePaths: [] }`) the output is
+ * byte-identical to the legacy single-arg `filterDiff(diff)`.
  */
-export declare function filterDiff(diff: string): FilterDiffResult;
+export declare function filterDiff(diff: string, options?: FilterDiffOptions): FilterDiffResult;
 //# sourceMappingURL=diff-filter.d.ts.map

package/dist/diff-filter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"diff-filter.d.ts","sourceRoot":"","sources":["../src/diff-filter.ts"],"names":[],"mappings":"AAsBA,sCAAsC;AACtC,MAAM,MAAM,UAAU,GAClB,UAAU,GACV,WAAW,GACX,UAAU,GACV,SAAS,GACT,iBAAiB,GACjB,aAAa,CAAA;AAEjB,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,UAAU,CAAA;CACnB;AAED,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAA;IACZ,UAAU,EAAE,MAAM,CAAA;IAClB,MAAM,EAAE,UAAU,CAAA;CACnB;AAED,MAAM,WAAW,gBAAgB;IAC/B,+DAA+D;IAC/D,QAAQ,EAAE,MAAM,CAAA;IAChB,uCAAuC;IACvC,YAAY,EAAE,WAAW,EAAE,CAAA;IAC3B,0DAA0D;IAC1D,YAAY,EAAE,WAAW,EAAE,CAAA;CAC5B;AAID;;;;;;;;;;;GAWG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,GAAG,gBAAgB,CA0DzD"}
+{"version":3,"file":"diff-filter.d.ts","sourceRoot":"","sources":["../src/diff-filter.ts"],"names":[],"mappings":"AAgCA,sCAAsC;AACtC,MAAM,MAAM,UAAU,GAClB,UAAU,GACV,WAAW,GACX,UAAU,GACV,SAAS,GACT,iBAAiB,GACjB,aAAa,GAGb,SAAS,CAAA;AAEb,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,UAAU,CAAA;CACnB;AAED,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAA;IACZ,UAAU,EAAE,MAAM,CAAA;IAClB,MAAM,EAAE,UAAU,CAAA;CACnB;AAED,MAAM,WAAW,gBAAgB;IAC/B,+DAA+D;IAC/D,QAAQ,EAAE,MAAM,CAAA;IAChB,uCAAuC;IACvC,YAAY,EAAE,WAAW,EAAE,CAAA;IAC3B,0DAA0D;IAC1D,YAAY,EAAE,WAAW,EAAE,CAAA;CAC5B;AAED,uEAAuE;AACvE,MAAM,WAAW,iBAAiB;IAChC;;;;;;;OAOG;IACH,QAAQ,CAAC,EAAE,OAAO,CAAA;IAClB;;;;;OAKG;IACH,WAAW,CAAC,EAAE,MAAM,EAAE,CAAA;CACvB;AAID;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,GAAE,iBAAsB,GAAG,gBAAgB,CAgF1F"}

package/dist/diff-filter.js

@@ -8,8 +8,16 @@
 // Pure-logic — no I/O, no clock, no randomness, no new runtime dep. ESLint
 // `no-restricted-imports` on packages/drift-engine/src/**/*.ts forbids fs /
 // child_process / http / https / @anthropic-ai/sdk / openai / @langchain/* /
-// process.env. Path classification uses hand-written predicates — NO
-// `picomatch` (Story Dev Notes §"Path classification predicates").
+// process.env. The BUILT-IN noise classifiers (lockfile/generated/vendored/
+// fixture) use hand-written predicates — NO `picomatch` (Story Dev Notes
+// §"Path classification predicates").
+//
+// ignore_code_scope (user-configurable ignore paths) is the ONE place this
+// module reaches for picomatch — via the shared `isFileInDocScope` predicate
+// (sibling module, same package, no new external dep). User globs MUST use the
+// same picomatch@4 dialect as the CLI config + the Action array-filter so a
+// file the Skill ignores is the file the Action ignores (ADR-2026-06-01
+// dialect parity). The hand-written built-ins above are unrelated and stay.
 //
 // Exposed via `index.ts` (unlike the P3.7.1 relevance internals, which run
 // INSIDE `buildPrompt`). The gate for this filter lives at the CONSUMER —
@@ -19,6 +27,7 @@
 // `@delfini/drift-engine/src/...` imports). The default consumer path never
 // invokes this module, so `buildPrompt` output stays byte-identical and the
 // NFR44 snapshot gate stays green (NFR49(b) parity discipline).
+import { isFileInDocScope } from './doc-scope.js';
 // -- Public entry point ------------------------------------------------------
 /**
  * Filter a unified-diff string deterministically.
@@ -31,8 +40,16 @@
  * not lose surrounding context.
  *
  * Identical input → identical output (NFR46 reproducibility carries forward).
+ *
+ * `options.builtins` (default `true`) gates the built-in noise classifiers;
+ * `options.ignorePaths` adds user `ignore_code_scope` dropping on top. With
+ * the default options (`{ builtins: true, ignorePaths: [] }`) the output is
+ * byte-identical to the legacy single-arg `filterDiff(diff)`.
  */
-export function filterDiff(diff) {
+export function filterDiff(diff, options = {}) {
+    const builtins = options.builtins ?? true;
+    const ignorePaths = options.ignorePaths ?? [];
+    const hasIgnore = ignorePaths.length > 0;
     const droppedPaths = [];
     const droppedHunks = [];
     const files = parseDiffIntoFiles(diff);
@@ -41,6 +58,22 @@ export function filterDiff(diff) {
         keptParts.push(files.preamble);
     }
     for (const file of files.files) {
+        // ignore_code_scope wins over everything — the dev explicitly told Delfini
+        // this code path is out of bounds for drift, so it is dropped whole and
+        // reported as 'ignored' regardless of whether the built-ins would also have
+        // matched it. Shared picomatch@4 predicate → parity with the CLI config and
+        // the Action's array-filter.
+        if (hasIgnore && isFileInDocScope(file.path, ignorePaths)) {
+            droppedPaths.push({ path: file.path, reason: 'ignored' });
+            continue;
+        }
+        // Built-ins off (ignore-only pass): a non-ignored file passes through
+        // verbatim — no path-level or hunk-level noise dropping. Byte-fidelity via
+        // the original slice.
+        if (!builtins) {
+            keptParts.push(file.rawSlice);
+            continue;
+        }
         // Path-level drops — classified first, in priority order. Lockfiles win
         // over generated/vendored/fixture since the canonical lockfile names
         // never overlap those patterns in practice.

package/dist/index.d.ts

@@ -6,6 +6,6 @@ export { normalizeDocScope, validateDocScopeEntry, classifyEntry, isFileInDocSco
 export { filterDiff } from './diff-filter.js';
 export { rankedFillSections } from './relevance.js';
 export type { AnalysisInput, AnalysisResult, DocFile, Contradiction, Addition, ClarifyingQuestion, PRMetadata, Severity, BuildPromptOptions, } from './types.js';
-export type { DropReason, DroppedPath, DroppedHunk, FilterDiffResult, } from './diff-filter.js';
+export type { DropReason, DroppedPath, DroppedHunk, FilterDiffResult, FilterDiffOptions, } from './diff-filter.js';
 export type { DocSection, DroppedSection, RankedFillCandidate, RankedFillResult, } from './relevance.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AA0BA,OAAO,EAAE,WAAW,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAA;AACvE,OAAO,EAAE,oBAAoB,EAAE,MAAM,gBAAgB,CAAA;AACrD,OAAO,EAAE,oBAAoB,EAAE,MAAM,oBAAoB,CAAA;AACzD,OAAO,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AAI5C,OAAO,EACL,iBAAiB,EACjB,qBAAqB,EACrB,aAAa,EACb,gBAAgB,GACjB,MAAM,gBAAgB,CAAA;AAOvB,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAA;AAS7C,OAAO,EAAE,kBAAkB,EAAE,MAAM,gBAAgB,CAAA;AAEnD,YAAY,EACV,aAAa,EACb,cAAc,EACd,OAAO,EACP,aAAa,EACb,QAAQ,EACR,kBAAkB,EAClB,UAAU,EACV,QAAQ,EACR,kBAAkB,GACnB,MAAM,YAAY,CAAA;AAEnB,YAAY,EACV,UAAU,EACV,WAAW,EACX,WAAW,EACX,gBAAgB,GACjB,MAAM,kBAAkB,CAAA;AAEzB,YAAY,EACV,UAAU,EACV,cAAc,EACd,mBAAmB,EACnB,gBAAgB,GACjB,MAAM,gBAAgB,CAAA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AA0BA,OAAO,EAAE,WAAW,EAAE,oBAAoB,EAAE,MAAM,qBAAqB,CAAA;AACvE,OAAO,EAAE,oBAAoB,EAAE,MAAM,gBAAgB,CAAA;AACrD,OAAO,EAAE,oBAAoB,EAAE,MAAM,oBAAoB,CAAA;AACzD,OAAO,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AAI5C,OAAO,EACL,iBAAiB,EACjB,qBAAqB,EACrB,aAAa,EACb,gBAAgB,GACjB,MAAM,gBAAgB,CAAA;AAOvB,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAA;AAS7C,OAAO,EAAE,kBAAkB,EAAE,MAAM,gBAAgB,CAAA;AAEnD,YAAY,EACV,aAAa,EACb,cAAc,EACd,OAAO,EACP,aAAa,EACb,QAAQ,EACR,kBAAkB,EAClB,UAAU,EACV,QAAQ,EACR,kBAAkB,GACnB,MAAM,YAAY,CAAA;AAEnB,YAAY,EACV,UAAU,EACV,WAAW,EACX,WAAW,EACX,gBAAgB,EAChB,iBAAiB,GAClB,MAAM,kBAAkB,CAAA;AAEzB,YAAY,EACV,UAAU,EACV,cAAc,EACd,mBAAmB,EACnB,gBAAgB,GACjB,MAAM,gBAAgB,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@delfini/drift-engine",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "private": false,
   "license": "Apache-2.0",
   "repository": {