fix: pre-landing review fixes

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
This commit is contained in:
garrytanandcapy-ai[bot] committed 2026-11-03 02:41:20 +00:00
1 parent 7384ca0d48
commit 5255dd9c22
6 files changed
+25 -20

No files matched your search

+4 -4
View File
@@ -113,7 +113,8 @@ function plainWords(item: ContractItem): string {
}
}
export function renderDoneMeans(contract: SkillContract): string {
/** The list; checker names the check and waive commands, which only a tree that ships bin/gstack-skill-contract should mention. */
export function renderDoneMeans(contract: SkillContract, checker = fs.existsSync(path.join(ROOT, 'bin', 'gstack-skill-contract'))): string {
if (contract.items.length === 0) return '';
return [
'## Done means',
@@ -121,9 +122,8 @@ export function renderDoneMeans(contract: SkillContract): string {
'Finish only when each of these exists (contract v1):',
'',
...contract.items.map(item => '- ' + code(item.id) + ': ' + plainWords(item) + '.'),
'',
code('gstack-skill-contract check') + ' reads them from the transcript (docs/skill-routing-and-contracts.md). ' +
'For an item that truly does not apply, run ' + code('gstack-skill-contract waive <item> --reason "<why>"') + ' (this session only; it counts as a miss).',
...(checker ? ['', code('gstack-skill-contract check') + ' reads them from the transcript (docs/skill-routing-and-contracts.md). ' +
'For an item that truly does not apply, run ' + code('gstack-skill-contract waive <item> --reason "<why>"') + ' (this session only; it counts as a miss).'] : []),
].join('\n');
}