Skip to content

docs(sdd): add JiraToPr gate commands and rollup.config.js module mapping - #861

Open
mkesavan13 wants to merge 1 commit into
masterfrom
agent/fix-jiratopr-manifest-gates
Open

docs(sdd): add JiraToPr gate commands and rollup.config.js module mapping#861
mkesavan13 wants to merge 1 commit into
masterfrom
agent/fix-jiratopr-manifest-gates

Conversation

@mkesavan13

Copy link
Copy Markdown

Summary

  • add canonical compile and unit-test commands for JiraToPr Gate 1 and Gate 2
  • force NODE_ENV=test for the unit-test command because the JiraToPr pod runs with NODE_ENV=production
  • map root-level rollup.config.js to the existing styles module and canonical styles specification

Root cause

The SPARK-843492 workflow stopped during Spec Sync because rollup.config.js is a JavaScript product source outside every module path declared in .sdd/manifest.json. After that mapping is corrected, Gate 1 also requires the canonical commands.compile role; commands.build is not a Gate 1 alias. The pod environment additionally exports NODE_ENV=production, which makes Jest load React's production build and fail Storyshots using act().

Validation

  • JSON parsing: passed
  • JiraToPr Spec Sync resolver with the representative SPARK-843492 change set: passed; components, adapters, and styles mapped with no warnings
  • Gate 1 resolver: compile resolves to npm run build
  • Gate 2 resolver: unit-test resolves to NODE_ENV=test npm run test
  • npm run build on Node 20: passed
  • NODE_ENV=test npm run test on Node 20: 14 suites passed, 230 tests passed
  • npm run linter: passed with 0 errors and 20 existing warnings

@mkesavan13
mkesavan13 force-pushed the agent/fix-jiratopr-manifest-gates branch from 215de21 to 07729b3 Compare August 7, 2026 13:00
@mkesavan13 mkesavan13 changed the title fix: configure JiraToPr manifest gates docs(sdd): add JiraToPr gate commands and rollup.config.js module mapping Aug 7, 2026
@Kesari3008
Kesari3008 marked this pull request as ready for review August 7, 2026 15:07
Comment thread .sdd/manifest.json
},
{
"path": "src/styles/",
"source_paths": [

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this change also needed for the harness ?

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