Skip to content

Comments

feat: add GitHub Job Summaries to marketplace pipeline#11

Merged
terraboops merged 3 commits intomainfrom
feat/pr-job-summaries
Feb 20, 2026
Merged

feat: add GitHub Job Summaries to marketplace pipeline#11
terraboops merged 3 commits intomainfrom
feat/pr-job-summaries

Conversation

@terraboops
Copy link
Collaborator

Summary

  • Track skipped files (no frontmatter) in discoverMarkdownComponents() and propagate through discoverAllComponents()
  • Add markdown job summaries to discover, validate, and generate actions via $GITHUB_STEP_SUMMARY
  • PR authors see component counts, validation pass/fail, and marketplace.json changes directly in the Actions UI

Changes

File Change
scripts/src/discover-components.js Add skipped array to discovery output
scripts/dist/discover-components.cjs Rebuilt bundle
agentic-marketplace/discover/action.yml Counts table + skipped files warning
agentic-marketplace/validate/action.yml Pass/fail status + collapsible output
agentic-marketplace/generate/action.yml marketplace.json diff/preview
scripts/test/discover-components.test.js 3 tests for skipped array

No new files, permissions, or workflow changes.

Test plan

  • npm test — 26/26 pass
  • npm run build — dist rebuilt
  • Local discover-all against PR #42 and #43 — skipped array present
  • CI passes on this PR
  • Re-trigger CI on bc-llm-skills PRs #42/#43 to confirm summaries render

🤖 Generated with Claude Code

terraboops and others added 3 commits February 20, 2026 01:24
Track skipped files (no frontmatter) in discovery and write markdown
summaries to $GITHUB_STEP_SUMMARY in discover/validate/generate actions.
PR authors now see component counts, validation results, and marketplace
changes directly in the GitHub Actions UI.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The "No generator.config.toml or .json found" message was going to
stdout via console.log, polluting JSON output from discover-all.
CI's node -e JSON.parse then failed. Switch to console.error so
stdout stays clean for machine-readable output.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The 2>&1 merged stderr into the JSON output variable, causing
JSON.parse failures in summary generation. Capture stderr to a
temp file instead, show it in logs, and keep OUTPUT clean for parsing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@terraboops terraboops merged commit ede40ae into main Feb 20, 2026
4 checks passed
@terraboops terraboops deleted the feat/pr-job-summaries branch February 20, 2026 09:29
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