Skip to content

Allow report_ui builds with explicit input/output paths - #111

Open
digitalWestie wants to merge 1 commit into
Jigsaw-Code:mainfrom
CoslaDigital:report-ui-cli-paths
Open

Allow report_ui builds with explicit input/output paths#111
digitalWestie wants to merge 1 commit into
Jigsaw-Code:mainfrom
CoslaDigital:report-ui-cli-paths

Conversation

@digitalWestie

@digitalWestie digitalWestie commented Sep 3, 2026

Copy link
Copy Markdown

Summary

  • Make src/report_ui invocable with explicit input/output paths (e.g. --bridging_scores, --summary, --output / --outputDir) so pipelines can build reports without copying artefacts into input/.
  • Preserve the existing copy-into-input/ workflow, default output layout (output/static, output/inline), dev / preview / github tasks, and the zero-install npx-based build steps.
  • Document the optional CLI flags additively in src/report_ui/README.md and briefly in the root README Step 6.

Test plan

  • Legacy: copy fixtures into input/, run npm run inlineoutput/inline/index.html
  • Explicit: node build.js inline --bridging_scores … --summary … --output …/new_report_build.html
  • Confirmed legacy and explicit inline HTML are byte-identical on the same fixtures
  • npm run static still writes output/static/index.html + quotes.json
  • Explicit-path build does not wipe the default output/ tree
  • Optional: smoke npm run dev / npm run preview locally

Keep the existing copy-into-input workflow and npx-based pipeline as the default, while adding optional CLI flags so automation can pass bridging scores, summary JSON, and output locations without staging files under input/.
@digitalWestie

Copy link
Copy Markdown
Author

I've tested this manually confirming we get the same outputs using the input folder and the flags method.

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