Files
msd-core/sdk/src/prompt-sanitizer.ts
Quang Do d4767ac2e0 fix: replace /gsd: slash command format with /gsd- skill format in all user-facing content (#1579)
* fix: replace /gsd: command format with /gsd- skill format in all suggestions

All next-step suggestions shown to users were still using the old colon
format (/gsd:xxx) which cannot be copy-pasted as skills. Migrated all
occurrences across agents/, commands/, get-shit-done/, docs/, README files,
bin/install.js (hardcoded defaults for claude runtime), and
get-shit-done/bin/lib/*.cjs (generate-claude-md templates and error messages).
Updated tests to assert new hyphen format instead of old colon format.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: migrate remaining /gsd: format to /gsd- in hooks, workflows, and sdk

Addresses remaining user-facing occurrences missed in the initial migration:

- hooks/: fix 4 user-facing messages (pause-work, update, fast, quick)
  and 2 comments in gsd-workflow-guard.js
- get-shit-done/workflows/: fix 21 Skill() literal calls that Claude
  executes directly (installer does not transform workflow content)
- sdk/prompt-sanitizer.ts: update regex to strip /gsd- format in addition
  to legacy /gsd: format; update JSDoc comment
- tests/: update autonomous-ui-steps, prompt-sanitizer to assert new format

Note: commands/gsd/*.md frontmatter (name: gsd:xxx) intentionally unchanged
— installer derives skillName from directory path, not the name field.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(plan-phase): preserve --chain flag in auto-advance sync and handle ui-phase gate in chain mode

Bug 1: step 15 sync-flag check only guarded against --auto, causing
_auto_chain_active to be cleared when plan-phase is invoked without
--auto in ARGUMENTS even though a --chain pipeline was active. Added
--chain to the guard condition, matching discuss-phase behaviour.

Bug 2: UI Design Contract gate (step 5.6) always exited the workflow
when UI-SPEC was missing, breaking the discuss --chain pipeline
silently. When _auto_chain_active is true, the gate now auto-invokes
gsd-ui-phase --auto via Skill() and continues to step 6 without
prompting. Manual invocations retain the existing AskUserQuestion flow.

* fix: remove <sub>/clear</sub> pattern and duplicate old-format command in discuss-phase.md

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-04 07:24:31 -04:00

72 lines
2.5 KiB
TypeScript

/**
* Prompt sanitizer — strips interactive CLI patterns from GSD-1 prompts
* so they're safe for headless SDK use.
*
* Patterns removed:
* - @file:... references (file injection directives)
* - /gsd-... skill commands
* - AskUserQuestion(...) calls
* - STOP directives in interactive contexts
* - SlashCommand() calls
* - 'wait for user' / 'ask the user' instructions
*/
// ─── Pattern definitions ─────────────────────────────────────────────────────
/**
* Each pattern is a regex that matches a full line (or inline span) to remove.
* We strip matching lines entirely to avoid leaving blank gaps that break
* markdown structure.
*/
const LINE_PATTERNS: RegExp[] = [
// @file:path/to/something references — entire line
/^.*@file:\S+.*$/gm,
// /gsd-command references — entire line containing a skill command
/^.*\/gsd[:-]\S+.*$/gm,
// AskUserQuestion(...) calls — entire line
/^.*AskUserQuestion\s*\(.*$/gm,
// SlashCommand() calls — entire line
/^.*SlashCommand\s*\(.*$/gm,
// STOP directives — lines that are primarily "STOP" instructions
// Match lines where STOP is used as an imperative (not as part of normal prose)
/^.*\bSTOP\b(?:\s+(?:and\s+)?(?:wait|ask|here|now)).*$/gm,
/^\s*STOP\s*[.!]?\s*$/gm,
// 'wait for user' / 'ask the user' instruction lines
/^.*\bwait\s+for\s+(?:the\s+)?user\b.*$/gim,
/^.*\bask\s+the\s+user\b.*$/gim,
];
// ─── Public API ──────────────────────────────────────────────────────────────
/**
* Strip interactive CLI patterns from a prompt string.
*
* Removes lines matching known interactive patterns (file references,
* slash commands, user-interaction directives) while preserving all
* other content unchanged.
*
* @param input - Raw prompt string, possibly containing interactive patterns
* @returns Cleaned prompt with interactive patterns removed
*/
export function sanitizePrompt(input: string): string {
if (!input) return input;
let result = input;
for (const pattern of LINE_PATTERNS) {
// Reset lastIndex for global regexes
pattern.lastIndex = 0;
result = result.replace(pattern, '');
}
// Collapse runs of 3+ blank lines down to 2 (preserve paragraph breaks)
result = result.replace(/\n{3,}/g, '\n\n');
return result.trim();
}