Skip to content

fix(goal-audit): commit empty-fixture dir; guard append-run-log JSON; document --fix alias - #474

Merged
cobusgreyling merged 1 commit into
cobusgreyling:mainfrom
pxmpsdev:fix/goal-audit-test-fixture-and-cli-docs-2
Aug 10, 2026
Merged

fix(goal-audit): commit empty-fixture dir; guard append-run-log JSON; document --fix alias#474
cobusgreyling merged 1 commit into
cobusgreyling:mainfrom
pxmpsdev:fix/goal-audit-test-fixture-and-cli-docs-2

Conversation

@pxmpsdev

@pxmpsdev pxmpsdev commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Guard invalid JSON in scripts/append-run-log.mjs (print Usage and exit 1 instead of stacktrace), document the already-implemented --fix alias in goal-audit --help, and track the empty fixture directory via .gitkeep.

Verification:

  • node --check ok for both files
  • invalid JSON -> Usage exit 1
  • goal-audit --help now shows --fix, --suggest and --fix produce identical output
  • --json/--md ok, fixtures-empty correctly scores G0
  • npm test in tools/goal-audit -> 2/2 pass

Small, focused fix with no breaking changes.

@pxmpsdev
pxmpsdev requested a review from cobusgreyling as a code owner August 9, 2026 20:50

@cobusgreyling cobusgreyling left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @pxmpsdev — clean, focused fix with clear verification. Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants