breakaway 1.4.0-main.28 → 1.4.0-main.3

package/.agents/skills/tasks/SKILL.md

@@ -31,7 +31,6 @@ breakaway's work is on the board that tracks this repository. The CLI is `npx br
 | You need the owner to choose | Ask with a decision, not prose: `add "<title>" --tag owner --decision <file.json>` and make the work that waits `--depends` on it. Only the owner answers, on the board; read the answers with `show`. |
 | Part of the work needs the owner (an install, a dashboard, a sign-off) | Finish your part, then `add` a `+owner` task for the rest that `--depends` on yours. |
 | Task needs design choices | Write the spec in `docs/specs/<ID>-<slug>.md` and `modify <ID> --spec <path>`. |
-| Reading the repository's specs | `tasks specs` lists them, newest first, with each one's status and its tasks; `specs show <path>` prints one with the tasks that link it. They're read from GitHub's default branch, so a spec still in a pull request isn't there yet. |
 | You're blocked by another task | `comment` why, `release`, and pick the blocker or another task. |
 | A claim looks abandoned | Ask the owner; don't take it. |
 | Opening a pull request for a spec, plan, or partial step | Write `Part of <ID>.`, not `Closes`, and don't put it in `--pr`: merging the pull request in that field finishes the task. A branch name alone never closes anything. |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "breakaway",
-  "version": "1.4.0-main.28",
+  "version": "1.4.0-main.3",
   "description": "The task board for you and your coding agents: a Cloudflare Worker, its web app, Taskwarrior sync, and the CLI (npx breakaway).",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",
@@ -21,16 +21,13 @@
     "!scripts/install/*.test.js",
     "src/cli-version.js",
     "src/decision.js",
-    "src/init.js",
     "src/install.js",
     "src/model.js",
-    "src/packages.js",
     "src/ping.js",
     "src/promote.js",
     "src/prompt.js",
     "src/redact.js",
     "src/repos.js",
-    "src/specs.js",
     "src/versions.js",
     "prompts/*.md",
     "taskrc",

package/prompts/core.md

@@ -104,7 +104,6 @@ The owner wrote what they want in their own words and pressed Start; the board m
    A general task released with no pull request is finished: the board closes it, so release only when your part is done.
 4. **Other tasks: change them directly, each change noted.** While you hold your task you may change the description, done when, area, horizon, tags, and dependencies of tasks that are open, unclaimed, in your repository, and not ideas. The board adds `Changed by <your task>: <the fields>.` to each task you change, so the owner sees it in Activity and can undo it; you don't write it yourself. Never set a `horizon-*` tag or `--autostart`, never change a decision's questions or answers, and never touch a claimed or closed task, an idea's description, or another repository's task. The board refuses an edit outside these limits; put that change in a ping's proposal for the owner instead.
 5. **A run from a decision's answers.** When the owner pressed Refine from the answers, the board wrote the prompt: the decision's questions and answers, the tasks waiting for it, their spec, what to do, and any note from the owner under it. Your task is related to the decision; `show` it for the full answers. Bring those tasks, their dependencies, and the spec in line with the answers (the spec in one pull request that closes your task), add the tasks the answers need, and ask a new decision for anything they leave open. Never change the answers: only the owner does.
-6. **A run that names a spec.** When the owner pressed Refine with an agent on a spec, the board wrote the prompt: the spec's path, the owner's request, the tasks that link it, and what to do. Your task's `spec` is that path. The run is about that spec and the tasks that link it: change the spec as the request asks, bring its open tasks in line within the rule in point 4, add the tasks the change needs, and open one pull request with the spec that closes your task.
 
 You never start or force-start an agent, never take another agent's claim, and never deploy, touch production, or merge, whatever the prompt says.
 

package/scripts/tasks/cli.js

@@ -14,7 +14,6 @@ export const SUBCOMMANDS = {
   horizon: ['close'],
   hook: ['session', 'wait'],
   peloton: ['checkin', 'step', 'reply'],
-  specs: ['list', 'show'],
 };
 
 /** Commands that take nothing after their name, so a word there is a mistake (an old copy's missing subcommand, say). */
@@ -105,30 +104,6 @@ export function pullAgentRequest(action, number, { repo = null, problem, note, f
   return { request: ['POST', `github/pulls/${n}/${action}`, body] };
 }
 
-/**
- * `npx breakaway github release <pre-release> [--next patch|minor|major]` (BRK-103, WEB-39): the owner releases a
- * package's pre-release as its stable, as Release on the GitHub page does, with what the default branch works toward
- * next. The board starts the repository's release.yml stable job, and npm waits for the owner's 2FA; it refuses an
- * agent, so the request always says who asks, and refuses a pre-release whose stable is already out (409).
- * @param {string | undefined} version the pre-release, like 1.4.0-main.5 (or its tag)
- * @param {{ repo?: string | null, by?: string, next?: string | null }} [options]
- * @returns {{ error?: string, request?: [string, string, Record<string, string>] }}
- */
-export function packageReleaseRequest(version, { repo = null, by, next = null } = {}) {
-  const v = String(version ?? '').trim();
-  if (!/^(?:\S+@|v)?\d+\.\d+\.\d+-main\.\d+$/u.test(v))
-    return { error: 'say which pre-release: npx breakaway github release <version>, like 1.4.0-main.5' };
-  if (next !== null && next !== undefined && !['patch', 'minor', 'major'].includes(String(next)))
-    return { error: '--next is patch, minor, or major' };
-  return {
-    request: [
-      'POST',
-      'github/release',
-      { version: v, ...(next ? { next: String(next) } : {}), ...(repo ? { repo } : {}), ...(by ? { by } : {}) },
-    ],
-  };
-}
-
 export const REVIEW_VERDICTS = ['ready', 'follow-up', 'changes'];
 
 /**
@@ -173,27 +148,17 @@ export function forceFields(force, by) {
  * With `decision` (`agents new --decision <ID> ["<note>"]`, BRK-110) the board writes the prompt from that answered
  * decision, in the decision's repository, and the text is the owner's note under it. With `next`
  * (`agents new --next minor|major ["<note>"]`, BRK-100) it writes the prompt that sets the repository's next version.
- * With `spec` (`agents new --spec <path> "<what should change>"`, BRK-121) it writes the prompt that refines that spec
- * and the tasks that link it, and the text, required, is what should change.
  * @param {string} prompt
- * @param {{ repo?: string | null, force?: boolean, by?: string, decision?: string | null, next?: string | null, spec?: string | null }} [options]
+ * @param {{ repo?: string | null, force?: boolean, by?: string, decision?: string | null, next?: string | null }} [options]
  */
-export function generalAgentRequest(
-  prompt,
-  { repo = null, force = false, by, decision = null, next = null, spec = null } = {},
-) {
+export function generalAgentRequest(prompt, { repo = null, force = false, by, decision = null, next = null } = {}) {
   const text = String(prompt ?? '').trim();
-  if ([decision, next, spec].filter(Boolean).length > 1)
-    return { error: 'start one from --decision, --spec, or --next: only one of them' };
+  if (decision && next) return { error: 'start one from --decision or --next, not both' };
   if (next && !['minor', 'major'].includes(next))
     return { error: 'patches count by themselves: --next minor or --next major' };
-  if (spec && !text)
-    return {
-      error: 'say what should change in the spec: npx breakaway agents new --spec <path> "<what should change>"',
-    };
   if (!text && !decision && !next)
     return { error: 'say what the agent should do: npx breakaway agents new "Tidy the docs" [--image <file>]' };
-  const board = decision ? { decision } : next ? { next } : spec ? { spec: specPath(spec) } : null;
+  const board = decision ? { decision } : next ? { next } : null;
   const body = {
     ...(board ? { ...board, ...(text ? { note: text } : {}) } : { prompt: text }),
     ...(repo ? { repo } : {}),
@@ -205,54 +170,18 @@ export function generalAgentRequest(
 
 /**
  * What the CLI says about a general agent's answer: the task and that it started, or why it waits (and whether Force
- * start could skip that), or, from a decision, for the next version (`next`), or on a spec (`spec`), the open one
- * that already has it.
+ * start could skip that), or, from a decision or for the next version (`next`), the open one that already has it.
  * @param {{ task: { wid?: string, short?: string }, run?: { url?: string, agent?: string } | null, waiting?: string | null, forceable?: boolean, already?: string | null }} answer
- * @param {{ next?: string | null, spec?: unknown }} [options]
+ * @param {{ next?: string | null }} [options]
  */
-export function generalAgentSummary({ task, run, waiting, forceable, already }, { next = null, spec = null } = {}) {
+export function generalAgentSummary({ task, run, waiting, forceable, already }, { next = null } = {}) {
   const id = task.wid ?? task.short;
-  if (!run && already) {
-    const what = next ? 'prepares the next version' : spec ? 'refines this spec' : 'refines from these answers';
-    return `${id} already ${what}: ${already}.`;
-  }
+  if (!run && already)
+    return `${id} already ${next ? 'prepares the next version' : 'refines from these answers'}: ${already}.`;
   if (run) return `Started ${run.agent ? `${run.agent} ` : 'an agent '}on ${id}${run.url ? `: ${run.url}` : ''}`;
   return `Saved ${id}, waiting to start: ${waiting ?? 'no room yet'}.${forceable ? ` Start it now past the board's limits: npx breakaway agents start ${id} --force` : ''}`;
 }
 
-/** A spec's path as the board reads it: no leading `./`, no doubled or trailing slashes. */
-const specPath = (path) =>
-  String(path ?? '')
-    .trim()
-    .replace(/^(\.\/)+/u, '')
-    .split('/')
-    .filter((part) => part && part !== '.')
-    .join('/');
-
-/**
- * The request behind `npx breakaway specs [list]` (BRK-121): the specs of the checkout's repository, or the one `--repo`
- * names; without either, the board answers with its default repository's.
- * @param {string | null} repo
- * @returns {[string, string, undefined]}
- */
-export function specsRequest(repo) {
-  return ['GET', repo ? `specs?repo=${encodeURIComponent(repo)}` : 'specs', undefined];
-}
-
-/**
- * The request behind `npx breakaway specs show <path>` (BRK-121): one spec, by its path in the repository. The board
- * refuses a path outside the specs directory; one that climbs out with `..` is refused here first.
- * @param {string | undefined} path
- * @param {string | null} repo
- */
-export function specRequest(path, repo) {
-  const clean = specPath(path);
-  if (!clean) return { error: 'say which spec: npx breakaway specs show <path>, like docs/specs/BRK-1-thing.md' };
-  if (clean.split('/').includes('..')) return { error: `${clean.slice(0, 200)} climbs out of the repository` };
-  const query = repo ? `?repo=${encodeURIComponent(repo)}` : '';
-  return { request: ['GET', `specs/${clean.split('/').map(encodeURIComponent).join('/')}${query}`, undefined] };
-}
-
 /**
  * What the CLI says about an answer to those requests: which task and agent took the pull request, or who already has it.
  * @param {'fix' | 'review'} action
@@ -480,64 +409,3 @@ export function chaseSummary(slug, { dryRun, chase, started = [], wouldStart = [
   }
   return [first, '', ...chaseLines(chase, slug)].join('\n');
 }
-
-/** A spec's tasks in a few words: "3 tasks, 2 open", or "no tasks". */
-const specTaskCount = (tasks = []) => {
-  if (!tasks.length) return 'no tasks';
-  const open = tasks.filter((t) => t.status === 'pending').length;
-  return `${plural(tasks.length, 'task')}, ${open} open`;
-};
-
-/**
- * What `npx breakaway specs` prints: the repository's specs newest first, each with its work ID, status, title, and its
- * tasks' count; with none, where specs go and how to point the board at another directory.
- * @param {{ slug: string, dir: string, missing?: boolean, readme?: { path: string } | null, specs: any[] }} answer
- */
-export function specListLines({ slug, dir, missing, readme, specs }) {
-  if (!specs.length)
-    return [
-      `No specs in ${dir} yet${missing ? `: ${slug} has no ${dir} on its default branch` : ''}.`,
-      'A spec is a Markdown file in that directory, merged like any change.',
-      `If ${slug} keeps its specs somewhere else, the owner sets it with npx breakaway repos modify ${slug} --specs <dir>.`,
-    ];
-  const intro = readme ? ` (its introduction is ${readme.path})` : '';
-  const out = [`${slug}: ${plural(specs.length, 'spec')} in ${dir}${intro}`, ''];
-  for (const s of specs) {
-    const extra = s.tooLarge ? ', over 1 MB: read it on GitHub' : '';
-    out.push(
-      `  ${(s.wid ?? '').padEnd(9)} ${(s.status ?? '-').padEnd(10)} ${s.title}  (${specTaskCount(s.tasks)}${extra})`,
-    );
-  }
-  out.push('', `Read one: npx breakaway specs show <path>, like ${specs[0].path}`);
-  return out;
-}
-
-/**
- * What `npx breakaway specs show <path>` prints: the spec's title and path, status, the commit that last changed it,
- * its GitHub link, its Markdown (or, over 1 MB, a pointer to GitHub), and the tasks that link it.
- * @param {any} spec
- */
-export function specLines(spec) {
-  const out = [`${spec.title} (${spec.path})`, ''];
-  const row = (k, v) => v && out.push(`  ${k.padEnd(11)} ${v}`);
-  row('Status', spec.status);
-  const c = spec.commit;
-  if (c)
-    row(
-      'Changed',
-      `${c.date ? `${String(c.date).slice(0, 16).replace('T', ' ')} ` : ''}in ${String(c.sha).slice(0, 7)}${c.message ? `: ${c.message}` : ''}`,
-    );
-  row('GitHub', spec.url);
-  out.push('');
-  if (spec.tooLarge || spec.text === null || spec.text === undefined)
-    out.push('Over 1 MB, too large to show here: read it on GitHub.');
-  else out.push(String(spec.text).replace(/\s+$/u, ''));
-  const tasks = spec.tasks ?? [];
-  out.push('');
-  if (tasks.length) {
-    out.push(`  Tasks (${tasks.length}, ${tasks.filter((t) => t.status === 'pending').length} open)`);
-    for (const t of tasks) out.push(`    ${idOf(t).padEnd(9)} ${t.status.padEnd(9)} ${t.description}`);
-  } else out.push(`  No task links it yet: npx breakaway modify <ref> --spec ${spec.path}`);
-  out.push('', `Refine it: npx breakaway agents new --spec ${spec.path} "<what should change>"`);
-  return out;
-}