opencode-swarm 7.159.3 → 7.160.1

package/dist/hooks/curator.d.ts

@@ -24,6 +24,7 @@
  * This dual dispatch means agent lists are incomplete — they capture factory-dispatched
  * curators but omit hook-dispatched ones. This is by design for hook-internal operations.
  */
+import { observeCuratorCompliance } from '../health/learning-health';
 import { checkRecommendations, recordEmittedRecommendations } from '../services/recommendation-ledger.js';
 import { listSkills, parseDraftFrontmatter, retireOrMarkStale, retireSkill } from '../services/skill-generator.js';
 import { getSkillVersion, reviseSkill } from '../services/skill-reviser.js';
@@ -51,6 +52,7 @@ export declare const _internals: {
     checkPhaseCompliance: typeof checkPhaseCompliance;
     normalizeAgentName: typeof normalizeAgentName;
     autoRetireSkills: typeof autoRetireSkills;
+    observeCuratorCompliance: typeof observeCuratorCompliance;
     /**
      * Retained deliberately (issue #2038 §7). The three curator decision sites
      * now read through `readSkillUsageEntriesWithCoverage` because a coverage

package/dist/hooks/final-context-accounting.d.ts

@@ -43,6 +43,8 @@ interface FinalAccountingOptions {
     /** Same seam createContextBudgetHandler uses: resolves the
      * configured target model for an agent name. */
     resolveAgentModelFn?: (agentName: string) => string | undefined;
+    /** Project directory (#2044): scopes the model-limit health observation. */
+    directory?: string;
 }
 /**
  * One-shot-per-band warning suppression (#2107 §3), backed by the

package/dist/hooks/model-limits.d.ts

@@ -25,6 +25,8 @@
  * live derivation replaced. They exist so a first-turn `messages.transform`
  * consumer degrades to a plausible number instead of the flat 128 000 floor.
  */
+import { type ContextWindowSource } from '../config/context-window';
+import { observeModelLimitResolution } from '../health/learning-health';
 /**
  * Native model context limits (in tokens) when used on their native platform.
  *
@@ -85,21 +87,46 @@ export declare function extractModelInfo(messages: MessageWithParts[]): {
  * @returns the sessionID, or `undefined` when no message carries one.
  */
 export declare function extractSessionId(messages: MessageWithParts[] | undefined): string | undefined;
+/**
+ * Coarse provenance class for a resolved model limit (issue #2044 item 1).
+ * Maps the fine-grained {@link ContextWindowSource} rung onto the five classes
+ * the issue names: `host` (the live `model.limit.context`), `override` (any
+ * user-authored `model_limits` entry), `provider_cap` / `native` (the two
+ * known-stale static tables), and `fallback` (the flat 128k default).
+ */
+export type ModelLimitSource = 'host' | 'override' | 'provider_cap' | 'native' | 'fallback';
+/** Provenance-bearing result of {@link resolveModelLimit} (issue #2044). */
+export interface ModelLimitResolution {
+    /** The resolved context limit in tokens. */
+    limit: number;
+    /** Coarse provenance class — the issue's enum. */
+    source: ModelLimitSource;
+    /** Which fine-grained resolution rung produced the limit. */
+    resolution: ContextWindowSource;
+}
+export declare function classifyModelLimitSource(resolution: ContextWindowSource): ModelLimitSource;
 /**
  * Static-table lookup used as `fallbackLookup` for
- * {@link resolveContextWindow}. Preserves the historical precedence of
- * the two tables relative to each other (provider cap first, then a
- * longest-prefix native match), and is reachable only when no live
- * `model.limit.context` was available.
+ * {@link resolveContextWindow}. Preserves the historical precedence of the two
+ * tables relative to each other (provider cap first, then a longest-prefix
+ * native match), and says WHICH table matched so the resolution source can
+ * distinguish `static_provider_cap` from `static_native` (issue #2044).
+ * Reachable only when no live `model.limit.context` was available.
  */
-export declare function lookupStaticModelLimit(modelID?: string, providerID?: string): number | undefined;
+export declare function lookupStaticModelLimit(modelID?: string, providerID?: string): {
+    tokens: number;
+    table: 'provider_cap' | 'native';
+} | undefined;
 /**
- * Resolves the context limit for a given model/provider combination.
+ * Resolves the context limit for a given model/provider combination WITH
+ * provenance (issue #2044).
  *
  * Thin adapter over the single derivation in `src/config/context-window.ts` —
  * this function exists so the `experimental.chat.messages.transform` consumers
- * (`context-budget.ts`, `knowledge-injector.ts`, `tools/context-status.ts`)
- * keep their existing `(modelID, providerID, overrides)` call shape. All
+ * (`context-budget.ts`, `knowledge-injector.ts`, `tools/context-status.ts`,
+ * `final-context-accounting.ts`) keep their existing
+ * `(modelID, providerID, overrides)` call shape while receiving the typed
+ * `{ limit, source }` provenance the observability series requires. All
  * ordering logic lives in the shared resolver; see its module header.
  *
  * Resolution order (first usable value wins):
@@ -111,44 +138,48 @@ export declare function lookupStaticModelLimit(modelID?: string, providerID?: st
  *    match (both known-stale; see the module header)
  * 6. `DEFAULT_MODEL_CONTEXT_TOKENS` (128000)
  *
- * Two ordering changes vs. the pre-#1619 implementation, both deliberate:
- * - `configOverrides.default` moved from below the static tables to above
- *   them. An explicitly authored user value losing to a hardcoded table was a
- *   defect; it is now consistent with the compound and model-only keys, which
- *   already outranked the tables.
- * - The live `model.limit.context` was inserted above both tables, which is
- *   the entire point of this change.
- *
- * Any value that is not a finite number ≥ 1000 is skipped rather than used, so
- * a malformed override or a catalog entry with `limit.context: 0` can never
- * produce a `NaN` / `Infinity` percentage.
+ * Any value that is not a finite number ≥ 1000 (untrusted rungs) / ≥ 1 (user
+ * rungs) is SKIPPED, never coerced; skipped user overrides are surfaced via a
+ * bounded `invalid_override_skipped` observation instead of silently
+ * disappearing (issue #2044 item 2).
  *
  * @param modelID - The model identifier (e.g., "claude-sonnet-4-6", "gpt-5")
  * @param providerID - The provider identifier (e.g., "github-copilot", "anthropic")
  * @param configOverrides - User configuration overrides (`context_budget.model_limits`)
  * @param liveContextLimit - The live `model.limit.context` recorded for this
  *   session by the `system.transform` hook, when one has been seen
- * @returns The resolved context limit in tokens
+ * @returns The resolved limit with coarse source class and fine-grained rung
  *
  * @example
  * // Live value wins over the stale static tables
  * resolveModelLimit("claude-sonnet-4-6", "github-copilot", {}, 200000)
- * // Returns: 200000
+ * // Returns: { limit: 200000, source: 'host', resolution: 'live_model_limit' }
  *
  * @example
  * // Explicit user override beats the live value
  * resolveModelLimit("gpt-5", "github-copilot", { "github-copilot/gpt-5": 60000 }, 400000)
- * // Returns: 60000
+ * // Returns: { limit: 60000, source: 'override', resolution: 'user_provider_model' }
  *
  * @example
  * // No live value: falls back to the static table (prefix match)
  * resolveModelLimit("claude-sonnet-4-6-20260301", "anthropic", {})
- * // Returns: 200000
+ * // Returns: { limit: 200000, source: 'native', resolution: 'static_native' }
  *
  * @example
  * // Malformed live value is ignored, not divided by
  * resolveModelLimit(undefined, undefined, {}, 0)
- * // Returns: 128000
+ * // Returns: { limit: 128000, source: 'fallback', resolution: 'static_default' }
  */
-export declare function resolveModelLimit(modelID?: string, providerID?: string, configOverrides?: Record<string, number>, liveContextLimit?: unknown): number;
+export declare function resolveModelLimit(modelID?: string, providerID?: string, configOverrides?: Record<string, number>, liveContextLimit?: unknown, 
+/** Project directory (threaded from the consumers) — scopes the #2044
+ * fallback-health observation to the owning project. */
+directory?: string): ModelLimitResolution;
+/**
+ * Bounded observation seam (invariant-7 DI): routes the model-limit fallback
+ * fact into the learning-health registry. Tests replace this to avoid touching
+ * global health state; production uses the real observer.
+ */
+export declare const _internals: {
+    observeResolution: typeof observeModelLimitResolution;
+};
 export {};

package/dist/hooks/pr-workflow-gate.d.ts

@@ -3,7 +3,7 @@ import { z } from 'zod';
 import { type CandidateSeverity, type FindingsSeverity, type RowFormatFamily } from '../background/candidate-contract.js';
 import { type LaneOutputArtifact } from '../background/lane-output-store.js';
 import { type BackgroundDelegationRecord, type BackgroundDelegationResult, type BackgroundDelegationWorkflowLaneFailureClass, type BackgroundDelegationWorkflowLaneRecovery, findByBatchId, sweepStaleDelegations } from '../background/pending-delegations.js';
-import { PR_REVIEW_BASE_DIMENSION_IDS, type PrReviewRiskImpact, type PrReviewRiskTag } from '../background/pr-review-contract.js';
+import { PR_REVIEW_BASE_DIMENSION_IDS, type PrReviewLaneResultEnvelope, type PrReviewRiskImpact, type PrReviewRiskTag } from '../background/pr-review-contract.js';
 import { type PrReviewInlineTriggerRow } from '../background/pr-review-trigger-contract.js';
 import { type RevisionDigestResult, resolveCommitCountSince, resolveCommitCountSinceAsync, resolveCurrentGitHead, resolveCurrentGitHeadAsync, resolveCurrentLocalHeadRefAsync, resolveCurrentUpstreamPushTarget, resolveCurrentUpstreamPushTargetAsync, resolveCurrentUpstreamRemoteRef, resolveExactRemoteBranchHead, resolveExactRemoteBranchHeadAsync, resolveIsExactSingleChildCommit, resolveIsExactSingleChildCommitAsync, resolveIsWorkingTreeClean, resolveIsWorkingTreeCleanAsync, resolvePrReviewDiffStats, resolvePrReviewDiffStatsAsync, resolvePrWorkflowRevisionDigest, resolvePrWorkflowRevisionDigestDetailed, resolveRemoteRefsContainingHead, resolveRemoteRefsContainingHeadAsync, resolveRemoteUrlIdentity, resolveRemoteUrlIdentityAsync } from '../background/workspace-snapshot.js';
 import { type PrReviewResilienceConfig } from '../config/schema.js';
@@ -50,6 +50,9 @@ export interface PrReviewDiscoveryLaneValidationInput {
         prHeadSha: string;
         gitHead: string;
         revisionDigest: string;
+        workflowInstanceId?: string;
+        workflowRevision?: number;
+        baseSha?: string;
         reviewScope?: string;
         checkWorkflowLane?: boolean;
     };
@@ -666,6 +669,25 @@ export declare function activatePrWorkflow(directory: string, sessionID: string,
     prUrl?: string;
 }): Promise<PrWorkflowGateState>;
 export declare function readPrWorkflowGateState(directory: string, sessionID: string): Promise<PrWorkflowGateState | null>;
+export type SubmitPrReviewResultOutcome = {
+    status: 'recorded' | 'duplicate';
+    receiptDigest: string;
+} | {
+    status: 'rejected';
+    reason: string;
+};
+/**
+ * Publish one structured base/micro discovery result from the exact child
+ * session that owns the live delegation. Lock order is deliberately
+ * workflow-session -> delegation-evidence; clear/abort takes the same outer
+ * lock and terminalization takes the same inner lock.
+ */
+export declare function submitPrReviewResult(directory: string, childSessionId: string, input: {
+    batchId: string;
+    laneId: string;
+    revisionDigest: string;
+    result: PrReviewLaneResultEnvelope;
+}): Promise<SubmitPrReviewResultOutcome>;
 /**
  * Serialize a pre-bind checkout-preservation controller with every terminal
  * PR-workflow transition for the raw parent session. The caller's action may
@@ -1128,6 +1150,7 @@ export declare function assertPrFeedbackGatePhaseSettled(directory: string, sess
 export declare function assertPrFeedbackReadyToPublish(directory: string, sessionID: string): Promise<PrWorkflowGateState>;
 /** Build the evidence join for a freshly armed generation. */
 declare function buildPublicationEvidenceJoin(state: PrWorkflowGateState, currentDigest: string): PrFeedbackPublicationEvidenceJoin | null;
+declare function boundPublicationDiagnostic(text: string): string;
 /**
  * Snapshot of the publication container with bounded history trimming.
  * History keeps the last {@link MAX_PUBLICATION_HISTORY_GENERATIONS}
@@ -1379,6 +1402,7 @@ export declare const _test_exports: {
     parseCanonicalCandidateRows: typeof parseCanonicalCandidateRows;
     resolvePrReviewRowFamily: typeof resolvePrReviewRowFamily;
     pipeFieldsCapped: typeof pipeFieldsCapped;
+    boundPublicationDiagnostic: typeof boundPublicationDiagnostic;
     indexVerdictRows: typeof indexVerdictRows;
     reviewerVerdictRowDigest: typeof reviewerVerdictRowDigest;
     workflowGateStateRelativePath: typeof workflowGateStateRelativePath;

package/dist/hooks/skill-usage-pending.d.ts

@@ -389,4 +389,4 @@ export declare function buildSkillUsageHealthPayload(doc: SkillUsagePendingDocum
 export declare function emitSkillUsageHealth(doc: SkillUsagePendingDocument, trigger: SkillUsageHealthPayload['trigger'], gauges: {
     bytes: number;
     limitBytes: number;
-}): void;
+}, directory?: string): void;