0b71657 · codetree · pear-git-v3

codetree commits / 0b71657

Parser: accept em-dash / en-dash as module-title separator

The module-spec parser's title regex required literal `--` (double
hyphen). Real specs often use `—` (em-dash U+2014) or `–` (en-dash
U+2013) — markdown renderers typosetter-style conversions, or authors
typing Unicode directly.

In pear-git-v3, 4 of 12 module specs use em-dash in their title
(anproto, hyperident, pgh, trust-map). Before this fix they silently
failed to parse, meaning codetree couldn't apply any fixes to them
(including high-confidence LOC drift and deprecation markers).

Regex updated from /^#\s+(\S+)\s+--/m to /^#\s+(\S+)\s+(?:--|—|–)\s+/m.

Impact: running codetree against pear-git-v3 after this fix:
  Before: 26 fixes detected (8 specs parsed)
  After:  41 fixes detected (12 specs parsed) — +15 from anproto,
          hyperident, pgh, trust-map now being visible to detectors.

Also:
- formatSuggest grew an `appliedFloor` param distinct from
  `confidenceFloor`, so `--suggest` shows ALL fixes (low/medium/high)
  while still labeling which are applicable at the current --apply
  floor. Previously --suggest with the default confidence 'high' hid
  all low-tier items.

96/96 tests pass unchanged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

0b716570e697ecbb26e7ef4285b21b55befe93b9

by plankatron <fleeky@gmail.com> · 2026-04-16T18:11:47.000Z

Changed files (3) +16 -6

index.js +4 -2

…	…	@@ -456,11 +456,13 @@ async function runSuggestOrApplyOrCheck (tree) {
456	456	return
457	457	}
458	458
459		// --suggest: print proposals
	459	// --suggest: print ALL proposals (the --confidence flag affects `result.applicable`
	460	// for --apply purposes; the printed suggestion view should show every tier so the
	461	// operator can see what MANUAL REVIEW items are queued up).
460	462	if (format === 'json') {
461	463	console.log(formatSuggestJson(result))
462	464	} else {
463		console.log(formatSuggest(result, { confidenceFloor: confidence }))
	465	console.log(formatSuggest(result, { confidenceFloor: 'low', appliedFloor: confidence }))
464	466	}
465	467	}
466	468

lib/doc-parser.js +6 -2

…	…	@@ -118,8 +118,12 @@ function parseModuleSpec (content, filename) {
118	118	replacedBy: null
119	119	}
120	120
121		// Module name from title: "# hyper-orgs -- Module Spec"
122		const titleMatch = content.match(/^#\s+(\S+)\s+--/m)
	121	// Module name from title. Accepts both "--" (double-hyphen) and "—"
	122	// (em-dash) as separators — both are common in hand-written specs and
	123	// markdown renderers often auto-convert one to the other.
	124	// "# hyper-orgs -- Module Spec"
	125	// "# hyperident — Module Spec"
	126	const titleMatch = content.match(/^#\s+(\S+)\s+(?:--\|—\|–)\s+/m)
123	127	if (titleMatch) spec.name = titleMatch[1]
124	128
125	129	// Location

lib/suggest-fixes.js +6 -2

…	…	@@ -462,10 +462,14 @@ function todayIso () {
462	462
463	463	/**
464	464	* Human-readable suggest output.
	465	*
	466	* opts.confidenceFloor — tier floor for DISPLAY (default 'low' = show all)
	467	* opts.appliedFloor — tier floor used by --apply (for labeling sections)
465	468	*/
466	469	export function formatSuggest (result, opts = {}) {
467	470	const lines = []
468	471	const show = opts.confidenceFloor \|\| 'low' // show everything by default
	472	const appliedFloor = opts.appliedFloor \|\| opts.confidenceFloor \|\| 'high'
469	473
470	474	const byConfidence = {
471	475	high: result.fixes.filter(f => f.confidence === 'high'),
…	…	@@ -479,7 +483,7 @@ export function formatSuggest (result, opts = {}) {
479	483	'(' + byConfidence.high.length + ' high, ' +
480	484	byConfidence.medium.length + ' medium, ' +
481	485	byConfidence.low.length + ' low).')
482		lines.push('Applicable at --confidence ' + (opts.confidenceFloor \|\| 'high') +
	486	lines.push('Applicable at --confidence ' + appliedFloor +
483	487	': ' + result.applicable.length)
484	488	lines.push('')
485	489
…	…	@@ -487,7 +491,7 @@ export function formatSuggest (result, opts = {}) {
487	491	if (byConfidence[tier].length === 0) continue
488	492	if (!ge(tier, show)) continue
489	493
490		const isAuto = ge(tier, opts.confidenceFloor \|\| 'high')
	494	const isAuto = ge(tier, appliedFloor)
491	495	const heading = tier === 'high'
492	496	? '## High-confidence fixes (auto-applicable)'
493	497	: tier === 'medium'