fix(isa-utils): function replacer in appendDecisionRow so ISA dollar amounts don't backref-expand

[alexfry-axeai]

What

appendDecisionRow (in LifeOS/install/hooks/lib/isa-utils.ts) appends the auto-rewind Decisions row using a string replacement built from the existing Decisions body:

return content.replace(decisionsRe, `${match[1]}${match[2].trimEnd()}\n${row}\n${match[3]}`);

The bug

That Decisions body routinely contains literal dollar amounts. String.prototype.replace interprets $1, $2, $&, etc. in a replacement string as regex backreferences — so an amount like $14,000 is read as capture group 1 followed by 4,000. On any resume of a completed ISA, the rewrite expands those into the captured section text, duplicating the ## Decisions heading and shredding the section (a $14,000 becomes <group-1>4,000, with fragment lines and repeated headers).

The fix

Use a function replacer, whose return value is inserted verbatim with no $ interpretation:

return content.replace(decisionsRe, () => `${match[1]}${match[2].trimEnd()}\n${row}\n${match[3]}`);

One line of behavior change, plus a short explanatory comment. appendDecisionRow is exported so it can be unit-tested.

Test

Adds isa-utils.test.ts, a regression test using synthetic $1/$2/$3/$4-prefixed amounts. It asserts every amount survives intact, exactly one ## Decisions heading remains, and the rewind row lands under Decisions. Fails on the old string replacement, passes on the fix.

bun test LifeOS/install/hooks/lib/isa-utils.test.ts   # 1 pass

[danielmiessler]

Ported with credit, test included. Closing, next release. Thanks.