@opengsd/gsd-core 1.2.0 → 1.3.0

package/get-shit-done/bin/lib/prompt-budget.cjs

@@ -1,399 +0,0 @@
-'use strict';
-
-/**
- * prompt-budget.cjs
- *
- * Pure functions for assembling and trimming review prompts to fit within
- * a token budget. Used by the review pipeline to support small-context models.
- *
- * Trim priority (in order — never violate):
- *   1. Instructions:   ALWAYS kept verbatim
- *   2. Reserve note tokens FIRST when any trim is anticipated
- *   3. Roadmap:        ALWAYS kept verbatim
- *   4. PROJECT.md:     head-shrink to projectMdHeadLines (default 40) if over budget
- *   5. Plans:          tail-truncate proportionally; never drop a whole plan
- *   6. Context:        DROP first if still over
- *   7. Research:       DROP second if still over
- *   8. Requirements:   DROP last (last-resort)
- *   9. Hard-fail:      if minimum-set exceeds effectiveBudget
- */
-
-const NOTE_RESERVE_TOKENS = 80;
-
-const DEFAULT_NOTE_TEMPLATE = [
-  '<note>',
-  'Prompt automatically trimmed to fit a {budget}-token budget.',
-  'Omitted sections: {omittedList}.',
-  'Plan content truncated by approximately {planTruncationPct}%.',
-  'Treat any missing context as out-of-scope rather than a review concern.',
-  '</note>',
-].join('\n');
-
-/**
- * Estimate tokens for a string. Chars / 4, rounded up.
- *
- * @param {string} text
- * @returns {number}
- */
-function estimateTokens(text) {
-  if (!text) return 0;
-  return Math.ceil(text.length / 4);
-}
-
-/**
- * Render the trim-disclosure note.
- *
- * @param {string} template
- * @param {number} budget
- * @param {string[]} omitted
- * @param {number} planTruncationPct
- * @returns {string}
- */
-function renderNote(template, budget, omitted, planTruncationPct) {
-  const omittedList = omitted.length > 0 ? omitted.join(', ') : 'none';
-  return template
-    .replace('{budget}', String(budget))
-    .replace('{omittedList}', omittedList)
-    .replace('{planTruncationPct}', String(Math.round(planTruncationPct)));
-}
-
-/**
- * Head-shrink a string to at most `maxLines` lines.
- *
- * @param {string} text
- * @param {number} maxLines
- * @returns {string}
- */
-function headShrink(text, maxLines) {
-  if (maxLines <= 0) return '';
-  let idx = -1;
-  let seen = 0;
-  while (seen < maxLines) {
-    idx = text.indexOf('\n', idx + 1);
-    if (idx === -1) return text;
-    seen += 1;
-  }
-  return text.slice(0, idx);
-}
-
-/**
- * Tail-truncate a string to at most `maxChars` characters.
- *
- * @param {string} text
- * @param {number} maxChars
- * @returns {string}
- */
-function tailTruncate(text, maxChars) {
-  if (text.length <= maxChars) return text;
-  return text.slice(0, maxChars);
-}
-
-/**
- * Assemble the final prompt string from its sections.
- *
- * @param {object} parts
- * @returns {string}
- */
-function assemblePrompt(parts) {
-  const {
-    instructions,
-    note,
-    roadmap,
-    projectMd,
-    plans,
-    context,
-    research,
-    requirements,
-  } = parts;
-
-  const blocks = [];
-
-  blocks.push(instructions);
-
-  if (note) blocks.push(note);
-
-  blocks.push('## Roadmap\n\n' + roadmap);
-
-  if (projectMd) blocks.push('## Project\n\n' + projectMd);
-
-  const planBlocks = plans
-    .map((p) => '### ' + p.file + '\n\n' + p.content)
-    .join('\n\n');
-  blocks.push('## Plans\n\n' + planBlocks);
-
-  if (context) blocks.push('## Context\n\n' + context);
-  if (research) blocks.push('## Research\n\n' + research);
-  if (requirements) blocks.push('## Requirements\n\n' + requirements);
-
-  return blocks.join('\n\n');
-}
-
-/**
- * Apply a token budget to a set of review prompt sections.
- * Returns the trimmed prompt and structured metadata.
- *
- * @param {object} param0
- * @param {object} param0.sections
- * @param {number} param0.budget
- * @param {object} [param0.options]
- * @returns {{ prompt: string, metadata: object }}
- */
-function applyBudget({ sections, budget, options = {} }) {
-  const {
-    safetyMarginPct = 10,
-    noteTemplate = DEFAULT_NOTE_TEMPLATE,
-    projectMdHeadLines = 40,
-  } = options;
-
-  const effectiveBudget = Math.floor(budget * (1 - safetyMarginPct / 100));
-
-  const {
-    instructions,
-    roadmap,
-    plans,
-    projectMd: projectMdRaw = null,
-    context: contextRaw = null,
-    research: researchRaw = null,
-    requirements: requirementsRaw = null,
-  } = sections;
-
-  // Working mutable state
-  let projectMd = projectMdRaw;
-  let context = contextRaw;
-  let research = researchRaw;
-  let requirements = requirementsRaw;
-  let workingPlans = plans.map((p) => ({ file: p.file, content: p.content }));
-
-  const omitted = [];
-  let projectMdShrunk = false;
-  let planTruncationPct = 0;
-  let noteInjected = false;
-  let hardFailed = false;
-
-  // Minimum-set check: instructions + roadmap + 1KB per plan.
-  // NOTE_RESERVE_TOKENS is intentionally excluded here: a note is only injected
-  // when trimming actually occurs, and a prompt that fits without any trim needs
-  // no note at all. Including NOTE_RESERVE_TOKENS here would cause false hard-fails
-  // for prompts that genuinely fit the effective budget untrimmed.
-  const MIN_PLAN_BYTES = 1024;
-  const minPlanTokens = plans.reduce((sum, p) => {
-    return sum + estimateTokens(p.content.slice(0, MIN_PLAN_BYTES));
-  }, 0);
-  const minSet =
-    estimateTokens(instructions) +
-    estimateTokens(roadmap) +
-    minPlanTokens;
-
-  if (minSet > effectiveBudget) {
-    return {
-      prompt: '',
-      metadata: {
-        budget,
-        effectiveBudget,
-        estimatedTokens: 0,
-        omitted: [],
-        projectMdShrunk: false,
-        planTruncationPct: 0,
-        hardFailed: true,
-        noteInjected: false,
-      },
-    };
-  }
-
-  // ── Budget accounting ──────────────────────────────────────────────────────
-  const TOKENS_ROADMAP_HEADER = estimateTokens('## Roadmap\n\n');
-  const TOKENS_PROJECT_HEADER = estimateTokens('## Project\n\n');
-  const TOKENS_PLANS_HEADER = estimateTokens('## Plans\n\n');
-  const TOKENS_CONTEXT_HEADER = estimateTokens('## Context\n\n');
-  const TOKENS_RESEARCH_HEADER = estimateTokens('## Research\n\n');
-  const TOKENS_REQUIREMENTS_HEADER = estimateTokens('## Requirements\n\n');
-  const TOKENS_PLAN_ITEM_HEADERS = workingPlans.reduce(
-    (sum, p) => sum + estimateTokens('### ' + p.file + '\n\n'),
-    0
-  );
-
-  const staticBaseTokens =
-    estimateTokens(instructions) +
-    TOKENS_ROADMAP_HEADER +
-    estimateTokens(roadmap) +
-    TOKENS_PLANS_HEADER +
-    TOKENS_PLAN_ITEM_HEADERS;
-
-  let projectTokens = projectMd
-    ? TOKENS_PROJECT_HEADER + estimateTokens(projectMd)
-    : 0;
-  let contextTokens = context
-    ? TOKENS_CONTEXT_HEADER + estimateTokens(context)
-    : 0;
-  let researchTokens = research
-    ? TOKENS_RESEARCH_HEADER + estimateTokens(research)
-    : 0;
-  let requirementsTokens = requirements
-    ? TOKENS_REQUIREMENTS_HEADER + estimateTokens(requirements)
-    : 0;
-  let planContentTokens = workingPlans.reduce(
-    (sum, p) => sum + estimateTokens(p.content),
-    0
-  );
-
-  const getCurrentBaseTokens = () =>
-    staticBaseTokens +
-    projectTokens +
-    planContentTokens +
-    contextTokens +
-    researchTokens +
-    requirementsTokens;
-
-  let currentBaseTokens = getCurrentBaseTokens();
-
-  // Detect budget pressure: is ANY trim needed?
-  // Pressure exists when the current base tokens already exceed the effective
-  // budget. Only when pressure is real do we reserve NOTE_RESERVE_TOKENS so
-  // the note itself fits after trimming. Checking against
-  // effectiveBudget - NOTE_RESERVE_TOKENS (the old threshold) would cause
-  // spurious pressure 80 tokens early, dropping sections that fit fine.
-  const baseTokens = currentBaseTokens;
-  const budgetUnderPressure = baseTokens > effectiveBudget;
-
-  // Available for content (reserve note slot when under pressure)
-  const contentBudget = budgetUnderPressure
-    ? effectiveBudget - NOTE_RESERVE_TOKENS
-    : effectiveBudget;
-
-  // ── Trim step 1: head-shrink PROJECT.md ───────────────────────────────────
-  if (currentBaseTokens > contentBudget && projectMd) {
-    const shrunk = headShrink(projectMd, projectMdHeadLines);
-    if (shrunk !== projectMd) {
-      projectMd = shrunk;
-      projectMdShrunk = true;
-      projectTokens = TOKENS_PROJECT_HEADER + estimateTokens(projectMd);
-      currentBaseTokens = getCurrentBaseTokens();
-    }
-  }
-
-  // ── Trim step 2: proportional plan truncation ─────────────────────────────
-  if (currentBaseTokens > contentBudget) {
-    // Compute tokens available for plan content only
-    const overhead =
-      staticBaseTokens +
-      projectTokens +
-      contextTokens +
-      researchTokens +
-      requirementsTokens;
-    const planBudgetTokens = contentBudget - overhead;
-    const totalPlanTokens = planContentTokens;
-
-    if (planBudgetTokens > 0 && planBudgetTokens < totalPlanTokens) {
-      // Proportional share per plan (at least 1KB per plan)
-      const totalOriginalChars = plans.reduce(
-        (sum, p) => sum + p.content.length,
-        0
-      );
-
-      const totalPlanCharsBudget = planBudgetTokens * 4;
-      workingPlans = workingPlans.map((p) => {
-        const proportionalShare =
-          totalOriginalChars > 0
-            ? Math.floor((p.content.length / totalOriginalChars) * totalPlanCharsBudget)
-            : 0;
-        const maxChars = Math.max(proportionalShare, MIN_PLAN_BYTES);
-        return { file: p.file, content: tailTruncate(p.content, maxChars) };
-      });
-
-      const newTotalChars = workingPlans.reduce(
-        (sum, p) => sum + p.content.length,
-        0
-      );
-      if (totalOriginalChars > 0) {
-        planTruncationPct =
-          ((totalOriginalChars - newTotalChars) / totalOriginalChars) * 100;
-      }
-      planContentTokens = workingPlans.reduce(
-        (sum, p) => sum + estimateTokens(p.content),
-        0
-      );
-      currentBaseTokens = getCurrentBaseTokens();
-    }
-  }
-
-  // ── Trim step 3: drop context ─────────────────────────────────────────────
-  if (currentBaseTokens > contentBudget && context) {
-    context = null;
-    omitted.push('context');
-    contextTokens = 0;
-    currentBaseTokens = getCurrentBaseTokens();
-  }
-
-  // ── Trim step 4: drop research ────────────────────────────────────────────
-  if (currentBaseTokens > contentBudget && research) {
-    research = null;
-    omitted.push('research');
-    researchTokens = 0;
-    currentBaseTokens = getCurrentBaseTokens();
-  }
-
-  // ── Trim step 5: drop requirements (last resort) ──────────────────────────
-  if (currentBaseTokens > contentBudget && requirements) {
-    requirements = null;
-    omitted.push('requirements');
-    requirementsTokens = 0;
-    currentBaseTokens = getCurrentBaseTokens();
-  }
-
-  // ── Decide whether note is actually needed ────────────────────────────────
-  const anyTrimOccurred =
-    omitted.length > 0 || projectMdShrunk || planTruncationPct > 0;
-
-  let note = null;
-  if (anyTrimOccurred) {
-    note = renderNote(noteTemplate, budget, omitted, planTruncationPct);
-    noteInjected = true;
-  }
-
-  // ── Assemble ──────────────────────────────────────────────────────────────
-  const prompt = assemblePrompt({
-    instructions,
-    note,
-    roadmap,
-    projectMd,
-    plans: workingPlans,
-    context,
-    research,
-    requirements,
-  });
-
-  const estimatedTokens = estimateTokens(prompt);
-
-  if (estimatedTokens > effectiveBudget) {
-    hardFailed = true;
-    return {
-      prompt: '',
-      metadata: {
-        budget,
-        effectiveBudget,
-        estimatedTokens,
-        omitted,
-        projectMdShrunk,
-        planTruncationPct,
-        hardFailed,
-        noteInjected,
-      },
-    };
-  }
-
-  return {
-    prompt,
-    metadata: {
-      budget,
-      effectiveBudget,
-      estimatedTokens,
-      omitted,
-      projectMdShrunk,
-      planTruncationPct,
-      hardFailed,
-      noteInjected,
-    },
-  };
-}
-
-module.exports = { estimateTokens, applyBudget };

package/get-shit-done/bin/lib/review-reviewer-selection.cjs

@@ -1,125 +0,0 @@
-'use strict';
-
-/**
- * Review Reviewer Selection Module
- *
- * Owns reviewer-selection policy projection for /gsd:review:
- * explicit flags > --all > review.default_reviewers > all detected.
- */
-
-const KNOWN_REVIEWER_SLUGS = [
-  'gemini',
-  'claude',
-  'codex',
-  'coderabbit',
-  'opencode',
-  'qwen',
-  'cursor',
-  'ollama',
-  'lm_studio',
-  'llama_cpp',
-];
-
-function normalizeConfiguredDefaultReviewers(rawValue) {
-  if (rawValue === undefined || rawValue === null) {
-    return { absent: true, values: [], errors: [] };
-  }
-  if (!Array.isArray(rawValue)) {
-    return {
-      absent: false,
-      values: [],
-      errors: ['review.default_reviewers must be a JSON array of reviewer slugs'],
-    };
-  }
-  if (rawValue.length === 0) {
-    return {
-      absent: false,
-      values: [],
-      errors: ['review.default_reviewers cannot be empty'],
-    };
-  }
-
-  const seen = new Set();
-  const normalized = [];
-  const errors = [];
-  for (const item of rawValue) {
-    if (typeof item !== 'string') {
-      errors.push('review.default_reviewers must contain only string slugs');
-      continue;
-    }
-    if (!/^[a-zA-Z0-9_-]+$/.test(item)) {
-      errors.push(`invalid reviewer slug in review.default_reviewers: ${item}`);
-      continue;
-    }
-    const slug = item.toLowerCase();
-    if (!seen.has(slug)) {
-      seen.add(slug);
-      normalized.push(slug);
-    }
-  }
-
-  return { absent: false, values: normalized, errors };
-}
-
-function resolveReviewerSelection(input) {
-  const detected = new Set((input.detected || []).map((v) => String(v).toLowerCase()));
-  const explicitFlags = new Set((input.explicitFlags || []).map((v) => String(v).toLowerCase()));
-  const allFlag = !!input.allFlag;
-  const normalizedDefaults = normalizeConfiguredDefaultReviewers(input.configuredDefaultReviewers);
-
-  const warnings = [];
-  const infos = [];
-  const errors = [...normalizedDefaults.errors];
-
-  let source = 'no_config_all_detected';
-  let selected = [];
-
-  if (explicitFlags.size > 0) {
-    source = 'explicit_flags';
-    selected = [...explicitFlags].filter((slug) => detected.has(slug));
-    const missing = [...explicitFlags].filter((slug) => !detected.has(slug));
-    if (missing.length > 0) {
-      infos.push(`explicit reviewers missing on host: ${missing.join(', ')}`);
-    }
-    if (selected.length === 0 && errors.length === 0) {
-      errors.push('no selected reviewers are available for explicit flags');
-    }
-  } else if (allFlag) {
-    source = 'all_flag';
-    selected = [...detected];
-  } else if (!normalizedDefaults.absent) {
-    source = 'config_default';
-    const knownDefaults = [];
-    for (const slug of normalizedDefaults.values) {
-      if (!KNOWN_REVIEWER_SLUGS.includes(slug)) {
-        warnings.push(`unknown reviewer slug in review.default_reviewers: ${slug}`);
-      } else {
-        knownDefaults.push(slug);
-      }
-    }
-    const undetected = knownDefaults.filter((slug) => !detected.has(slug));
-    if (undetected.length > 0) {
-      infos.push(`configured reviewers not detected on this host: ${undetected.join(', ')}`);
-    }
-    selected = knownDefaults.filter((slug) => detected.has(slug));
-    if (selected.length === 0 && errors.length === 0) {
-      errors.push('all configured default reviewers are unavailable on this host');
-    }
-  } else {
-    selected = [...detected];
-  }
-
-  return {
-    source,
-    selected: selected.sort(),
-    warnings,
-    infos,
-    errors,
-  };
-}
-
-module.exports = {
-  KNOWN_REVIEWER_SLUGS,
-  normalizeConfiguredDefaultReviewers,
-  resolveReviewerSelection,
-};

package/get-shit-done/bin/lib/roadmap-command-router.cjs

@@ -1,28 +0,0 @@
-'use strict';
-
-const { ROADMAP_SUBCOMMANDS } = require('./command-aliases.cjs');
-const { routeCjsCommandFamily } = require('./cjs-command-router-adapter.cjs');
-
-/**
- * Manifest-backed roadmap subcommand router.
- * Keeps gsd-tools.cjs thin while preserving existing command semantics.
- */
-function routeRoadmapCommand({ roadmap, args, cwd, raw, error }) {
-  routeCjsCommandFamily({
-    args,
-    subcommands: ROADMAP_SUBCOMMANDS,
-    unsupported: {},
-    error,
-    unknownMessage: (_subcommand, available) => `Unknown roadmap subcommand. Available: ${available.join(', ')}`,
-    handlers: {
-      'get-phase': () => roadmap.cmdRoadmapGetPhase(cwd, args[2], raw),
-      analyze: () => roadmap.cmdRoadmapAnalyze(cwd, raw),
-      'update-plan-progress': () => roadmap.cmdRoadmapUpdatePlanProgress(cwd, args[2], raw),
-      'annotate-dependencies': () => roadmap.cmdRoadmapAnnotateDependencies(cwd, args[2], raw),
-    },
-  });
-}
-
-module.exports = {
-  routeRoadmapCommand,
-};