@mjasnikovs/pi-task 0.16.4 → 0.17.0

package/dist/task/phases.d.ts

@@ -42,6 +42,14 @@ export interface PhaseConfig {
 export declare function extractToolingCommands(research: string): string[] | null;
 /** Replace the TOOLING section in a research string with a VERIFIED-TOOLING section. */
 export declare function replaceToolingWithVerified(research: string, verifiedCommands: string[]): string;
+/**
+ * Manifest + config content (orientation tiers 0–1) for refine, with the preserve
+ * directive. Reuses the SAME orientation machinery research feeds its workers — not
+ * a parallel reader — scoped to the "accumulative" files a from-scratch rewrite
+ * silently destroys. '' (non-git/empty/orientation-off) leaves refine unchanged, so
+ * a genuinely greenfield task is still free to create.
+ */
+export declare function refineExistingFilesBlock(deps: PhaseDeps): Promise<string>;
 export declare const phaseRefine: (deps: PhaseDeps, raw: string, planContext?: string) => Promise<string>;
 export declare function phaseVerifyTooling(deps: PhaseDeps, research: string): Promise<string>;
 export interface PhaseResearchDeps extends ExternalContextDeps {

package/dist/task/phases.js

@@ -11,7 +11,7 @@ import { search as defaultSearch } from '../workers/search-core.js';
 import { extractEnrichTargets } from './enrichment.js';
 import { isIntegrationUnknown } from './unknown-routing.js';
 import { getFileInventory } from './file-inventory.js';
-import { buildOrientation } from './orientation.js';
+import { buildOrientation, orientationTier } from './orientation.js';
 import { getConfig } from '../config/config.js';
 import { readFile } from 'node:fs/promises';
 import { resolve } from 'node:path';
@@ -63,15 +63,67 @@ export function replaceToolingWithVerified(research, verifiedCommands) {
     return replaced;
 }
 // ─── Phase functions ─────────────────────────────────────────────────────────
-export const phaseRefine = (deps, raw, planContext) => runPhaseWithLoopGuard(deps, 'refine', 'read', hint => prependHint(hint, appendNoThink(REFINE_PROMPT(raw, planContext))), 
-// refine's deliverable is a 4-section text rewrite that never strictly
-// needs a successful read — on a test-writing task against a large
-// existing codebase the model over-explores (re-reads source hunting for
-// the impl) and burns the loop budget. Degrade to a no-tools final
-// attempt instead of hard-failing the whole run. See TASK_0016 (mx5):
-// refine looped 3×/resume forever; the deliverable was always producible
-// from the title + design doc alone.
-{ degradeOnExhaustion: true });
+// Authoritative directive that travels with the refine orientation block. Refine
+// runs BEFORE research, so on a "Scaffold project with package.json…" title it has
+// no signal the file already exists and authors greenfield strip-constraints
+// ("bun-plugin-tailwind the only dependency", "exactly N scripts") that compose
+// then obeys over the research facts — the implementer executes a wholesale
+// rewrite that drops every existing dependency and script (mx5 TASK_0001 emptied
+// package.json; the REAL pipeline reproduced it 3/3 even with research surfacing
+// the deps). Handing refine the manifest/config CONTENT up front, with this
+// reframe, fixes it at the origin (A/B: 1/5 → 5/5 preserve, 5/5 end-to-end).
+const REFINE_PRESERVE_DIRECTIVE = 'EXISTING FILES ON DISK — AUTHORITATIVE (overrides any "scaffold / create / set '
+    + 'up / initialize / from scratch / only / exactly / minimal" wording in the task '
+    + 'below): the files shown in the PROJECT ORIENTATION block ALREADY EXIST. When '
+    + 'the task says to scaffold, create, set up, initialize, or configure a file that '
+    + 'already exists, your GOAL and CONSTRAINTS MUST frame it as an in-place UPDATE '
+    + 'that PRESERVES every existing dependency, devDependency, script, field, and '
+    + "compiler option — adding or changing only the task's explicit delta. Do NOT "
+    + 'author any constraint that empties, reduces to a fixed/minimal set, renames, '
+    + 'recreates, or drops existing entries (no "X is the only dependency", no '
+    + '"exactly N scripts", no "produce exactly N files from scratch"). An existing '
+    + 'entry is NEVER scope drift and is never removed because it "belongs to a later step".';
+/**
+ * Manifest + config content (orientation tiers 0–1) for refine, with the preserve
+ * directive. Reuses the SAME orientation machinery research feeds its workers — not
+ * a parallel reader — scoped to the "accumulative" files a from-scratch rewrite
+ * silently destroys. '' (non-git/empty/orientation-off) leaves refine unchanged, so
+ * a genuinely greenfield task is still free to create.
+ */
+export async function refineExistingFilesBlock(deps) {
+    if (!getConfig().orientation)
+        return '';
+    const inventoryRaw = await getFileInventory(deps.cwd, deps.signal).catch(() => '');
+    if (inventoryRaw.length === 0)
+        return '';
+    const paths = inventoryRaw
+        .split('\n')
+        .map(l => l.trim())
+        .filter(l => l.length > 0 && (orientationTier(l) === 0 || orientationTier(l) === 1));
+    if (paths.length === 0)
+        return '';
+    const { block } = await buildOrientation(paths, async (p) => {
+        try {
+            return await readFile(resolve(deps.cwd, p), 'utf8');
+        }
+        catch {
+            return null;
+        }
+    });
+    return block.trim().length === 0 ? '' : `${REFINE_PRESERVE_DIRECTIVE}\n\n${block.trim()}`;
+}
+export const phaseRefine = async (deps, raw, planContext) => {
+    const existingFiles = await refineExistingFilesBlock(deps).catch(() => '');
+    return runPhaseWithLoopGuard(deps, 'refine', 'read', hint => prependHint(hint, appendNoThink(REFINE_PROMPT(raw, planContext, existingFiles))), 
+    // refine's deliverable is a 4-section text rewrite that never strictly
+    // needs a successful read — on a test-writing task against a large
+    // existing codebase the model over-explores (re-reads source hunting for
+    // the impl) and burns the loop budget. Degrade to a no-tools final
+    // attempt instead of hard-failing the whole run. See TASK_0016 (mx5):
+    // refine looped 3×/resume forever; the deliverable was always producible
+    // from the title + design doc alone.
+    { degradeOnExhaustion: true });
+};
 export async function phaseVerifyTooling(deps, research) {
     const commands = extractToolingCommands(research);
     if (!commands || commands.length === 0) {

package/dist/task/prompts.d.ts

@@ -47,7 +47,7 @@ export declare const COMPRESS_LABEL_PROMPT: (title: string, maxChars: number) =>
  * "Scaffold …" title re-expands the entire design into one task (validated: a real
  * /task-auto run implemented all 24 steps under step 1).
  */
-declare const REFINE_PROMPT: (raw: string, planContext?: string) => string;
+declare const REFINE_PROMPT: (raw: string, planContext?: string, existingFiles?: string) => string;
 declare const RESEARCH_READ_ONLY_CONSTRAINT = "IMPORTANT: You are ONLY allowed to READ. Do NOT create, modify, or delete any files. Use the read, grep, find, and ls tools to inspect the repo.";
 declare const RESEARCH_FILES_PROMPT: (refined: string) => string;
 declare const RESEARCH_APIS_PROMPT: (refined: string) => string;

package/dist/task/prompts.js

@@ -61,7 +61,7 @@ ${title}`;
  * "Scaffold …" title re-expands the entire design into one task (validated: a real
  * /task-auto run implemented all 24 steps under step 1).
  */
-const REFINE_PROMPT = (raw, planContext) => `${planContext ? planContext + '\n\n---\n\n' : ''}You receive a user's task description for an AI coding agent. Rewrite it to be unambiguous and actionable.
+const REFINE_PROMPT = (raw, planContext, existingFiles) => `${planContext ? planContext + '\n\n---\n\n' : ''}You receive a user's task description for an AI coding agent. Rewrite it to be unambiguous and actionable.
 
 Output structure (four sections, exact headings, in this order):
 
@@ -87,7 +87,7 @@ Rules:
 - Do not invent requirements not implied by the input.
 - If the task references a design/spec document (an @-path or a named spec file), READ it and treat it as authoritative. Carry its concrete schema verbatim into GOAL/CONSTRAINTS — table and column names, types, endpoint methods and paths, enum values. The task title is only a pointer into that spec: where the title and the spec disagree, follow the spec, and never introduce a table, column, endpoint, or dependency the spec does not define.
 - Do not output any preamble, commentary, or markdown headings beyond the four sections above.
-
+${existingFiles && existingFiles.trim() ? `\n${existingFiles.trim()}\n` : ''}
 Task: ${raw}`;
 // ─── Research fan-out prompts ─────────────────────────────────────────────────
 const RESEARCH_READ_ONLY_CONSTRAINT = `IMPORTANT: You are ONLY allowed to READ. Do NOT create, modify, or delete any files. Use the read, grep, find, and ls tools to inspect the repo.`;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.16.4",
+    "version": "0.17.0",
     "description": "Deterministic spec-orchestration for local models, with a bundled real-time remote web view and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",