Skip to content

Commit 1995191

Browse files
tyaginidhiclaude
andcommitted
Wire --projectRoot into the remaining discover-site-components consumers
The site-referenced table discovery (this PR) made discover-site-components return customTables (and thus missing.customTables) ONLY when --projectRoot (or --datamodelManifest) is passed — without a local signal it returns [] rather than the old publisher-prefix dump. setup-solution Step D was updated to pass --projectRoot, but three other consumers that read missing.customTables were missed, so their "custom tables missing from the solution" completeness check silently reported 0 for every site: - export-solution Phase 2.5 (pre-export completeness) - plan-alm Phase 1 (pre-plan completeness) - deploy-pipeline Phase 3.5 (pre-sync completeness) Added --projectRoot "." to all three so the check is restored with the correct site-scoped count. (setup-solution Phase 5.4b/5.4c calls consume missing.envVars / missing.powerpagecomponents / siteLanguages, not customTables, so they're correct as-is with --publisherPrefix/--solutionId.) 1211 tests pass, alm-lint 0 findings. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent e8ad4c8 commit 1995191

3 files changed

Lines changed: 6 additions & 3 deletions

File tree

plugins/power-pages/skills/deploy-pipeline/SKILL.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,8 @@ node "${CLAUDE_PLUGIN_ROOT}/scripts/lib/discover-site-components.js" \
300300
--envUrl "{devEnvUrl}" --token "{DEV_TOKEN}" \
301301
--siteId "{websiteRecordId from .solution-manifest.json}" \
302302
--publisherPrefix "{publisherPrefix from .solution-manifest.json}" \
303-
--solutionId "{solutionId from .solution-manifest.json}"
303+
--solutionId "{solutionId from .solution-manifest.json}" \
304+
--projectRoot "."
304305
```
305306
306307
Parse stdout and evaluate `missing.*`. **Before doing anything else**, capture the **pre-sync state** so a post-sync re-confirmation gate can show what changed:

plugins/power-pages/skills/export-solution/SKILL.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,8 @@ node "${CLAUDE_PLUGIN_ROOT}/scripts/lib/discover-site-components.js" \
158158
--envUrl "{envUrl}" --token "{token}" \
159159
--siteId "{websiteRecordId}" \
160160
--publisherPrefix "{publisherPrefix from .solution-manifest.json}" \
161-
--solutionId "{solutionId}"
161+
--solutionId "{solutionId}" \
162+
--projectRoot "."
162163
```
163164

164165
Parse stdout and evaluate `missing`. **Before doing anything else**, capture the **pre-sync state** so a post-sync re-confirmation gate can show what changed:

plugins/power-pages/skills/plan-alm/SKILL.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -242,7 +242,8 @@ Steps:
242242
--envUrl "{envUrl}" --token "{token}" \
243243
--siteId "{websiteRecordId from powerpages.config.json}" \
244244
--publisherPrefix "{solutionManifest.publisher.prefix}" \
245-
--solutionId "{solutionManifest.solution.solutionId}"
245+
--solutionId "{solutionManifest.solution.solutionId}" \
246+
--projectRoot "."
246247
```
247248
248249
Parse stdout and evaluate `missing.*`:

0 commit comments

Comments
 (0)