Skip to content

Sydshields/dash 1732 turn existing templates in js files#95

Merged
syd-shields merged 2 commits into
test-productionfrom
sydshields/dash-1732-turn-existing-templates-in-js-files
May 13, 2026
Merged

Sydshields/dash 1732 turn existing templates in js files#95
syd-shields merged 2 commits into
test-productionfrom
sydshields/dash-1732-turn-existing-templates-in-js-files

Conversation

@syd-shields

@syd-shields syd-shields commented May 13, 2026

Copy link
Copy Markdown
Contributor

testing CI


Note

Medium Risk
Moderate risk because it changes how filtered builds clean javascript/ output and how package.json scripts.build is rewritten, which could unintentionally delete or alter generated template artifacts.

Overview
Refactors scripts/build-javascript.mjs to parse CLI filter flags once (parseBuildOptions) and reuse the resulting options for both file filtering and output cleanup, keeping partial rebuild behavior consistent when using include/exclude template or path flags.

Updates package.json adaptation to safely drop a leading compile-only tsc from scripts.build: it now removes the script only when it’s just tsc, or preserves any chained commands after &&/||/; so downstream steps (e.g. next build) remain.

Reviewed by Cursor Bugbot for commit 2a862c0. Bugbot is set up for automated code reviews on this repo. Configure here.

@syd-shields syd-shields merged commit 09b60a5 into test-production May 13, 2026
3 checks passed
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.

1 participant