@delfini/action-core 0.1.0

package/dist/github-client-shared.js

@@ -0,0 +1,240 @@
+import * as core from '@actions/core';
+import * as github from '@actions/github';
+import { readDocsAtHeadViaGitTrees } from './doc-reader.js';
+export function getPrContext() {
+    const { payload, repo } = github.context;
+    if (payload.pull_request) {
+        return {
+            owner: repo.owner,
+            repo: repo.repo,
+            pullNumber: payload.pull_request.number,
+            headSha: payload.pull_request.head.sha,
+            baseSha: payload.pull_request.base.sha,
+        };
+    }
+    if (payload.issue?.pull_request && payload.comment) {
+        const prUrl = payload.issue.pull_request.url;
+        return {
+            owner: repo.owner,
+            repo: repo.repo,
+            pullNumber: payload.issue.number,
+            headSha: '',
+            baseSha: '',
+            ...extractShasFromIssueComment(prUrl),
+        };
+    }
+    throw new Error(`Unsupported event: expected pull_request or issue_comment on a PR, got ${github.context.eventName}`);
+}
+export async function listChangedFiles(octokit, ctx) {
+    const files = [];
+    let page = 1;
+    while (true) {
+        const { data } = await octokit.rest.pulls.listFiles({
+            owner: ctx.owner,
+            repo: ctx.repo,
+            pull_number: ctx.pullNumber,
+            per_page: 100,
+            page,
+        });
+        if (data.length === 0) {
+            break;
+        }
+        for (const file of data) {
+            files.push({
+                filename: file.filename,
+                status: file.status,
+                patch: file.patch,
+            });
+        }
+        if (data.length < 100) {
+            break;
+        }
+        page++;
+    }
+    return files;
+}
+export async function getFileContent(octokit, ctx, path, ref) {
+    try {
+        const { data } = await octokit.rest.repos.getContent({
+            owner: ctx.owner,
+            repo: ctx.repo,
+            path,
+            ref,
+        });
+        if (Array.isArray(data) || data.type !== 'file' || !data.content) {
+            return null;
+        }
+        return Buffer.from(data.content, 'base64').toString('utf-8');
+    }
+    catch (error) {
+        if (typeof error === 'object' &&
+            error !== null &&
+            'status' in error &&
+            error.status === 404) {
+            return null;
+        }
+        throw error;
+    }
+}
+export async function readDocs(octokit, ctx, docScope, options = {}) {
+    // v6.1 — walk at the PR head, not the merge-base. After Approve and Commit
+    // splices doc updates onto the PR branch, the head ref carries those updates;
+    // walking at base would re-detect the same drift forever (and would force
+    // FR88f to keep creating new pending findings every re-run because section
+    // anchors are LLM-non-deterministic). Reading at head closes that loop:
+    // Approve and Commit -> head advances -> next walk sees the updated doc ->
+    // PASS verdict -> Stream 4a auto-supersede -> review resolves.
+    //
+    // Story 3.12 (ADR-2026-06-01) — Full mode now reads via ONE recursive
+    // git-trees call + the shared `isFileInDocScope` matcher (picomatch@4),
+    // exactly as Lite mode does (Story P2.6). `docScope` is the canonical
+    // `string[]` (multi-path/glob). The old per-directory `getContent` walk
+    // (`readDocsFromPath`) is retired in this story.
+    return readDocsAtHeadViaGitTrees(octokit, ctx.owner, ctx.repo, docScope, ctx.headSha, options);
+}
+// v6.0 thin top-level PR comment (Story 3.11). Uses issues.createComment /
+// updateComment (NOT pulls.createReview) — replaces the pre-v6.0 Request
+// Changes / Approve review primitive on success paths. Idempotent on `Bot`
+// author + label-prefix marker — re-runs update the same comment in place.
+//
+// The hidden HTML marker (DELFINI_PR_COMMENT_MARKER) is appended to every
+// v6.0 thin template body so the matcher can be specific without depending
+// on visible body text. A foreign Bot whose comment happens to start with
+// `**` (Dependabot, Renovate, etc.) is NOT matched because they don't carry
+// the marker. The matcher uses `body.includes(marker)` not `startsWith` so
+// the marker can ride at the end of the body without affecting layout.
+//
+// Pagination: caps at 5 pages (500 comments) before warn-and-create — same
+// shape as postOrUpdateReview's 100-cap warn pattern. PRs with >500 comments
+// are pathological; warn-and-create produces a benign duplicate at worst.
+const PR_COMMENT_PAGINATION_PAGE_CAP = 5;
+const PR_COMMENT_PAGINATION_PER_PAGE = 100;
+// Hidden HTML marker — invisible in rendered markdown, byte-stable in source.
+// Keep this constant in sync with the marker appended by emitStreamDecision.
+export const DELFINI_PR_COMMENT_MARKER = '<!-- delfini-pr-comment -->';
+export async function postOrUpdatePrComment(octokit, ctx, body, options) {
+    let existingId = null;
+    let extraMatches = 0;
+    let pagesScanned = 0;
+    for (let page = 1; page <= PR_COMMENT_PAGINATION_PAGE_CAP; page++) {
+        const { data } = await octokit.rest.issues.listComments({
+            owner: ctx.owner,
+            repo: ctx.repo,
+            issue_number: ctx.pullNumber,
+            per_page: PR_COMMENT_PAGINATION_PER_PAGE,
+            page,
+        });
+        pagesScanned += 1;
+        const matches = data.filter((c) => c.user?.type === 'Bot' && (c.body ?? '').includes(options.marker));
+        if (matches.length > 0) {
+            if (existingId === null) {
+                existingId = matches[0].id;
+                extraMatches = matches.length - 1;
+            }
+            else {
+                extraMatches += matches.length;
+            }
+            // Continue paginating to count duplicates across pages for the warn-on-many path.
+            // Stop early once we've scanned enough to be confident — first match is the
+            // one we'll update; extras just produce a warning.
+            if (data.length < PR_COMMENT_PAGINATION_PER_PAGE)
+                break;
+            continue;
+        }
+        if (data.length < PR_COMMENT_PAGINATION_PER_PAGE)
+            break;
+    }
+    if (extraMatches > 0) {
+        core.warning(`Delfini: ${extraMatches + 1} matching Delfini PR comments found — updating the first; consider manual cleanup of duplicates.`);
+    }
+    if (existingId === null &&
+        pagesScanned === PR_COMMENT_PAGINATION_PAGE_CAP) {
+        core.warning(`Delfini: PR comment scan capped at ${PR_COMMENT_PAGINATION_PAGE_CAP * PR_COMMENT_PAGINATION_PER_PAGE} comments — idempotency may misfire on this PR.`);
+    }
+    if (existingId !== null) {
+        try {
+            await octokit.rest.issues.updateComment({
+                owner: ctx.owner,
+                repo: ctx.repo,
+                comment_id: existingId,
+                body,
+            });
+            return;
+        }
+        catch (error) {
+            // 404 = comment was deleted between listComments and updateComment
+            // (manual delete by reviewer, race with another tool). Fall through to
+            // createComment so the PR doesn't end up with no Delfini comment at all.
+            if (isHttpStatus(error, 404)) {
+                core.warning(`Delfini: PR comment #${existingId} disappeared between scan and update — creating a fresh comment.`);
+            }
+            else {
+                throw error;
+            }
+        }
+    }
+    await octokit.rest.issues.createComment({
+        owner: ctx.owner,
+        repo: ctx.repo,
+        issue_number: ctx.pullNumber,
+        body,
+    });
+}
+function isHttpStatus(error, status) {
+    return (typeof error === 'object' &&
+        error !== null &&
+        'status' in error &&
+        error.status === status);
+}
+// 403 (read-only token) and 404 (resource not visible) both signal the
+// workflow lacks permission to write a check or PR comment — typically a fork
+// PR. Callers tolerate these (warn + continue) rather than crashing the run.
+// Story P2.2 (AC7) — moved here from pipeline.ts so the Full and Lite
+// pipelines share one predicate.
+export function isForbiddenError(error) {
+    return isHttpStatus(error, 403) || isHttpStatus(error, 404);
+}
+export async function createCheckStatus(octokit, ctx, conclusion, title, summary, detailsUrl) {
+    // Conditional spread keeps the request body free of an explicit
+    // `details_url: undefined` — Octokit treats absent and `undefined` fields
+    // differently for some endpoints, and we want GitHub's default Details
+    // affordance (the built-in check_run page) when no URL is provided.
+    //
+    // 'in_progress' is the sentinel for "attention needed, static yellow icon":
+    // we use the legacy Commit Statuses API with state='pending' because that
+    // renders a STATIC yellow dot (no spinner). The check_runs API has no
+    // conclusion that renders static yellow — status='in_progress' animates,
+    // conclusion='action_required' renders red, and conclusion='neutral' renders
+    // grey. Chromatic and other "needs review" integrations also use commit
+    // statuses for this exact reason. The 140-char description cap is enforced
+    // by GitHub; truncate defensively so a long title doesn't 422.
+    if (conclusion === 'in_progress') {
+        const description = title.length > 140 ? title.slice(0, 139) + '…' : title;
+        await octokit.rest.repos.createCommitStatus({
+            owner: ctx.owner,
+            repo: ctx.repo,
+            sha: ctx.headSha,
+            state: 'pending',
+            context: 'Delfini Docs Drift Check',
+            description,
+            ...(detailsUrl !== undefined ? { target_url: detailsUrl } : {}),
+        });
+        return;
+    }
+    await octokit.rest.checks.create({
+        owner: ctx.owner,
+        repo: ctx.repo,
+        head_sha: ctx.headSha,
+        name: 'Delfini Docs Drift Check',
+        status: 'completed',
+        conclusion,
+        output: { title, summary },
+        ...(detailsUrl !== undefined ? { details_url: detailsUrl } : {}),
+    });
+}
+function extractShasFromIssueComment(_prUrl) {
+    // SHAs are not available on issue_comment events directly.
+    // The caller must fetch them from the PR API after context creation.
+    // Returning empty strings signals that SHAs need to be resolved.
+    return {};
+}

package/dist/index.d.ts

@@ -0,0 +1,19 @@
+export { readDocsViaGitTrees, readDocsAtHeadViaGitTrees } from './doc-reader.js';
+export type { DocFile, DocsReadResult, ReadDocsOptions, ScopeSource } from './doc-reader.js';
+export { parseFrontMatter, stripFrontMatter } from './doc-exclusion.js';
+export type { ExcludedDoc, ExclusionReason, FrontMatterResult } from './doc-exclusion.js';
+export { buildAnalysisInput } from './analysis-input.js';
+export type { BuildAnalysisInputOptions } from './analysis-input.js';
+export { classifyPr } from './smart-skip.js';
+export type { SmartSkipOptions, SmartSkipResult } from './smart-skip.js';
+export { buildUnifiedDiff } from './diff-builder.js';
+export { DELFINI_PR_COMMENT_MARKER, createCheckStatus, getFileContent, getPrContext, isForbiddenError, listChangedFiles, postOrUpdatePrComment, readDocs, } from './github-client-shared.js';
+export type { ChangedFile, PrContext } from './github-client-shared.js';
+export { readPipelineInputs } from './pipeline-inputs.js';
+export type { Enforcement, PipelineDeps, PipelineInputs } from './pipeline-inputs.js';
+export { createOrchestrator } from './adapters/factory.js';
+export { SingleCallOrchestrator, loadTemplate as loadPromptTemplate, } from './adapters/single-call/orchestrator.js';
+export { createChatModel } from './adapters/single-call/model.js';
+export type { LLMProvider } from './adapters/single-call/model.js';
+export type { AnalysisOrchestrator } from './ports/orchestrator.js';
+//# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAWA,OAAO,EAAE,mBAAmB,EAAE,yBAAyB,EAAE,MAAM,iBAAiB,CAAA;AAChF,YAAY,EAAE,OAAO,EAAE,cAAc,EAAE,eAAe,EAAE,WAAW,EAAE,MAAM,iBAAiB,CAAA;AAE5F,OAAO,EAAE,gBAAgB,EAAE,gBAAgB,EAAE,MAAM,oBAAoB,CAAA;AACvE,YAAY,EAAE,WAAW,EAAE,eAAe,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAA;AAEzF,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAA;AACxD,YAAY,EAAE,yBAAyB,EAAE,MAAM,qBAAqB,CAAA;AAEpE,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAA;AAC5C,YAAY,EAAE,gBAAgB,EAAE,eAAe,EAAE,MAAM,iBAAiB,CAAA;AAExE,OAAO,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAA;AAEpD,OAAO,EACL,yBAAyB,EACzB,iBAAiB,EACjB,cAAc,EACd,YAAY,EACZ,gBAAgB,EAChB,gBAAgB,EAChB,qBAAqB,EACrB,QAAQ,GACT,MAAM,2BAA2B,CAAA;AAClC,YAAY,EAAE,WAAW,EAAE,SAAS,EAAE,MAAM,2BAA2B,CAAA;AAEvE,OAAO,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAA;AACzD,YAAY,EAAE,WAAW,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAA;AAErF,OAAO,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAA;AAC1D,OAAO,EACL,sBAAsB,EACtB,YAAY,IAAI,kBAAkB,GACnC,MAAM,wCAAwC,CAAA;AAC/C,OAAO,EAAE,eAAe,EAAE,MAAM,iCAAiC,CAAA;AACjE,YAAY,EAAE,WAAW,EAAE,MAAM,iCAAiC,CAAA;AAElE,YAAY,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAA"}

package/dist/index.js

@@ -0,0 +1,20 @@
+// @delfini/action-core — single barrel (project rule: named exports only;
+// barrel only at packages/*/src/index.ts).
+//
+// Story P3.9.2a (Lite/Full artifact split, Mechanism A): the shared
+// analysis-pipeline core of the Delfini GitHub Action, consumed by the public
+// Lite artifact (apps/action) and the Full artifact (apps/action-full;
+// re-homed into delfini-web at P3.9.4) — algorithm parity by construction.
+//
+// V1 stability posture: internal shared core, published for transparency and
+// delfini-web consumption — no semver API-stability promise (see README.md).
+export { readDocsViaGitTrees, readDocsAtHeadViaGitTrees } from './doc-reader.js';
+export { parseFrontMatter, stripFrontMatter } from './doc-exclusion.js';
+export { buildAnalysisInput } from './analysis-input.js';
+export { classifyPr } from './smart-skip.js';
+export { buildUnifiedDiff } from './diff-builder.js';
+export { DELFINI_PR_COMMENT_MARKER, createCheckStatus, getFileContent, getPrContext, isForbiddenError, listChangedFiles, postOrUpdatePrComment, readDocs, } from './github-client-shared.js';
+export { readPipelineInputs } from './pipeline-inputs.js';
+export { createOrchestrator } from './adapters/factory.js';
+export { SingleCallOrchestrator, loadTemplate as loadPromptTemplate, } from './adapters/single-call/orchestrator.js';
+export { createChatModel } from './adapters/single-call/model.js';

package/dist/pipeline-inputs.d.ts

@@ -0,0 +1,23 @@
+import type { GitHub } from '@actions/github/lib/utils';
+import type { AnalysisOrchestrator } from './ports/orchestrator.js';
+type Octokit = InstanceType<typeof GitHub>;
+export type Enforcement = 'required' | 'warning';
+export interface PipelineInputs {
+    docScope: string[];
+    enforcement: Enforcement;
+    githubToken: string;
+    /**
+     * Story P3.7.2 / FR151 — opt-in diff pre-filter. Default `false` — the
+     * assembled analysis input is byte-identical to the pre-story baseline.
+     * Optional in the type so test fixtures that build a minimal PipelineInputs
+     * keep compiling; `readPipelineInputs` always sets it explicitly.
+     */
+    enableDiffPreFilter?: boolean;
+}
+export interface PipelineDeps {
+    octokit: Octokit;
+    orchestrator?: AnalysisOrchestrator;
+}
+export declare function readPipelineInputs(): PipelineInputs;
+export {};
+//# sourceMappingURL=pipeline-inputs.d.ts.map

package/dist/pipeline-inputs.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pipeline-inputs.d.ts","sourceRoot":"","sources":["../src/pipeline-inputs.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAA;AAGvD,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAA;AAUnE,KAAK,OAAO,GAAG,YAAY,CAAC,OAAO,MAAM,CAAC,CAAA;AAE1C,MAAM,MAAM,WAAW,GAAG,UAAU,GAAG,SAAS,CAAA;AAEhD,MAAM,WAAW,cAAc;IAS7B,QAAQ,EAAE,MAAM,EAAE,CAAA;IAClB,WAAW,EAAE,WAAW,CAAA;IACxB,WAAW,EAAE,MAAM,CAAA;IACnB;;;;;OAKG;IACH,mBAAmB,CAAC,EAAE,OAAO,CAAA;CAC9B;AAED,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,OAAO,CAAA;IAChB,YAAY,CAAC,EAAE,oBAAoB,CAAA;CACpC;AAED,wBAAgB,kBAAkB,IAAI,cAAc,CA2CnD"}

package/dist/pipeline-inputs.js

@@ -0,0 +1,42 @@
+import * as core from '@actions/core';
+import { normalizeDocScope } from '@delfini/drift-engine';
+export function readPipelineInputs() {
+    // Story P2.7 — Lite-mode doc-scope input is `doc_scope` (canonical name
+    // shared with the drift-engine algebra, CLI, FR88g API response, DB column,
+    // and Web settings). ADR-2026-06-01 / Story P2.6 — `doc_scope` accepts a
+    // newline- or comma-delimited list (action.yml inputs are strings only). A
+    // single bare path still works. The split list is fed through
+    // `normalizeDocScope` from `@delfini/drift-engine` (POSIX, trim, trailing-
+    // slash strip, dedupe, `./` / `..` resolution all handled there — do NOT
+    // re-implement here).
+    //
+    // The default `['docs/']` keeps Lite's user-observable default identical to
+    // pre-P2.6 (single string `'docs/'`) — only the type changes. The defensive
+    // collapse-to-empty fallback covers the user-types-",,,," case so an
+    // accidentally-empty normalized list never silently turns into a zero-scope
+    // run (which would skip every PR via FR57(b)).
+    const rawDocScope = core.getInput('doc_scope');
+    const entries = rawDocScope.length > 0 ? rawDocScope.split(/[\n,]/) : ['docs/'];
+    const normalized = normalizeDocScope(entries);
+    // A non-empty input that normalises to nothing (e.g. "   ", ",,," , "\n\n")
+    // is almost certainly a typo, not an intentional default request. Warn so the
+    // operator can tell "I omitted doc_scope → default" apart from "my doc_scope
+    // collapsed → default" before silently analysing `docs/` (Story P2.6).
+    if (rawDocScope.length > 0 && normalized.length === 0) {
+        core.warning(`doc_scope "${rawDocScope}" contains no valid doc-scope entries after ` +
+            'normalisation — falling back to the default "docs/". Check for stray ' +
+            'delimiters or whitespace-only entries.');
+    }
+    // Fallback flows through `normalizeDocScope` so the collapse-to-default path
+    // yields the exact same value (`['docs']`) as the omitted-input path — no
+    // trailing-slash inconsistency between the two default routes.
+    const docScope = normalized.length > 0 ? normalized : normalizeDocScope(['docs/']);
+    const rawEnforcement = (core.getInput('enforcement') || 'warning').toLowerCase();
+    const enforcement = rawEnforcement === 'required' ? 'required' : 'warning';
+    const githubToken = process.env.GITHUB_TOKEN ?? core.getInput('github_token');
+    // Story P3.7.2 / FR151 — `enable_diff_prefilter` action.yml input. Default
+    // false: any value other than the literal "true" (case-insensitive) leaves
+    // the gate off so the pre-story behaviour is the path of least resistance.
+    const enableDiffPreFilter = core.getInput('enable_diff_prefilter').toLowerCase() === 'true';
+    return { docScope, enforcement, githubToken, enableDiffPreFilter };
+}

package/dist/ports/orchestrator.d.ts

@@ -0,0 +1,5 @@
+import type { AnalysisInput, AnalysisResult } from '@delfini/drift-engine';
+export interface AnalysisOrchestrator {
+    analyze(input: AnalysisInput): Promise<AnalysisResult>;
+}
+//# sourceMappingURL=orchestrator.d.ts.map

package/dist/ports/orchestrator.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"orchestrator.d.ts","sourceRoot":"","sources":["../../src/ports/orchestrator.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAA;AAE1E,MAAM,WAAW,oBAAoB;IACnC,OAAO,CAAC,KAAK,EAAE,aAAa,GAAG,OAAO,CAAC,cAAc,CAAC,CAAA;CACvD"}

package/dist/ports/orchestrator.js

@@ -0,0 +1 @@
+export {};

package/dist/smart-skip.d.ts

@@ -0,0 +1,10 @@
+export interface SmartSkipOptions {
+    docScope: string[];
+    skipTestFiles?: boolean;
+}
+export interface SmartSkipResult {
+    shouldSkip: boolean;
+    reason: string;
+}
+export declare function classifyPr(changedFiles: string[], options: SmartSkipOptions): SmartSkipResult;
+//# sourceMappingURL=smart-skip.d.ts.map

package/dist/smart-skip.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"smart-skip.d.ts","sourceRoot":"","sources":["../src/smart-skip.ts"],"names":[],"mappings":"AAEA,MAAM,WAAW,gBAAgB;IAS/B,QAAQ,EAAE,MAAM,EAAE,CAAA;IAClB,aAAa,CAAC,EAAE,OAAO,CAAA;CACxB;AAED,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,OAAO,CAAA;IACnB,MAAM,EAAE,MAAM,CAAA;CACf;AA8DD,wBAAgB,UAAU,CAAC,YAAY,EAAE,MAAM,EAAE,EAAE,OAAO,EAAE,gBAAgB,GAAG,eAAe,CA+C7F"}

package/dist/smart-skip.js

@@ -0,0 +1,85 @@
+import { isFileInDocScope } from '@delfini/drift-engine';
+const categoryLabels = {
+    dependency: 'dependency update',
+    ci: 'CI config change',
+    generated: 'generated file',
+    node_modules: 'node_modules change',
+    test: 'test file change',
+};
+const DEPENDENCY_BASENAMES = new Set([
+    'package.json',
+    'pnpm-lock.yaml',
+    'package-lock.json',
+    'yarn.lock',
+]);
+function basename(filePath) {
+    const parts = filePath.split('/');
+    return parts[parts.length - 1] ?? filePath;
+}
+function classifyFile(filePath, skipTestFiles) {
+    if (filePath.includes('node_modules/')) {
+        return 'node_modules';
+    }
+    if (DEPENDENCY_BASENAMES.has(basename(filePath))) {
+        return 'dependency';
+    }
+    if (filePath.startsWith('.github/') && (filePath.endsWith('.yml') || filePath.endsWith('.yaml'))) {
+        return 'ci';
+    }
+    if (filePath.endsWith('.generated.ts') || filePath.endsWith('.gen.ts')) {
+        return 'generated';
+    }
+    if (skipTestFiles &&
+        (filePath.endsWith('.test.ts') || filePath.endsWith('.test.tsx') || filePath.endsWith('.spec.ts'))) {
+        return 'test';
+    }
+    return null;
+}
+function formatReason(counts) {
+    const parts = [];
+    for (const [category, count] of counts) {
+        const label = categoryLabels[category];
+        parts.push(`${count} ${label}${count > 1 ? 's' : ''}`);
+    }
+    return parts.join(', ');
+}
+export function classifyPr(changedFiles, options) {
+    if (changedFiles.length === 0) {
+        return { shouldSkip: true, reason: 'No changed files detected' };
+    }
+    const skipTestFiles = options.skipTestFiles ?? false;
+    const counts = new Map();
+    let docInScopeCount = 0;
+    for (const filePath of changedFiles) {
+        // Story P2.6 — shared in-scope predicate (picomatch@4 dialect via
+        // `@delfini/drift-engine`). Replaces the legacy private `isDocFile`
+        // prefix check so smart-skip and the Lite reader's git-trees matcher
+        // can never silently disagree on the same scope.
+        if (isFileInDocScope(filePath, options.docScope)) {
+            docInScopeCount += 1;
+            continue;
+        }
+        const category = classifyFile(filePath, skipTestFiles);
+        if (category === null) {
+            return { shouldSkip: false, reason: 'Business-logic changes detected' };
+        }
+        counts.set(category, (counts.get(category) ?? 0) + 1);
+    }
+    let structurallyUninterestingCount = 0;
+    for (const count of counts.values()) {
+        structurallyUninterestingCount += count;
+    }
+    // FR57(b) v6.1 — every changed file is a doc-in-scope; smart-skip fires.
+    if (docInScopeCount > 0 && structurallyUninterestingCount === 0) {
+        const noun = docInScopeCount === 1 ? 'change' : 'changes';
+        return { shouldSkip: true, reason: `${docInScopeCount} doc-only ${noun} in doc scope` };
+    }
+    // FR57(a) — pure structurally-uninteresting changes; existing skip path.
+    if (docInScopeCount === 0) {
+        return { shouldSkip: true, reason: formatReason(counts) };
+    }
+    // Mixed doc-in-scope + structurally-uninteresting. The two skip checks
+    // are independent (FR57 v6.1) — a mixed PR satisfies neither, so analysis
+    // runs.
+    return { shouldSkip: false, reason: 'Mixed doc and non-doc changes detected' };
+}

package/package.json

@@ -0,0 +1,41 @@
+{
+  "name": "@delfini/action-core",
+  "version": "0.1.0",
+  "private": false,
+  "license": "Apache-2.0",
+  "publishConfig": {
+    "access": "public"
+  },
+  "type": "module",
+  "description": "Shared analysis-pipeline core of the Delfini GitHub Action (doc reader, smart-skip, analysis-input assembly, single-call orchestrator adapters, shared GitHub client, pipeline input reader). Internal shared core published for transparency and delfini-web consumption — no semver API-stability promise in V1.",
+  "exports": {
+    ".": {
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.js"
+    }
+  },
+  "files": [
+    "dist",
+    "README.md"
+  ],
+  "dependencies": {
+    "@actions/core": "^1.11.1",
+    "@actions/github": "^6.0.0",
+    "@langchain/anthropic": "^1.3.29",
+    "@langchain/core": "^1.1.45",
+    "@langchain/openai": "^1.4.5",
+    "gray-matter": "^4.0.3",
+    "@delfini/drift-engine": "0.1.0"
+  },
+  "devDependencies": {
+    "@types/node": "^22.10.2",
+    "typescript": "^5.7.2",
+    "vitest": "^3.0.5"
+  },
+  "scripts": {
+    "build": "tsc -b && node scripts/copy-prompt.mjs",
+    "typecheck": "tsc --noEmit",
+    "lint": "eslint src",
+    "test": "vitest run"
+  }
+}