Skip to content

chore: restore official BMad skill rendering runtime - #127

Merged
chuanxu742-glitch merged 1 commit into
2233admin:mainfrom
chuanxu742-glitch:chuanxu742-glitch/bmad-runtime-restoration
Sep 6, 2026
Merged

chore: restore official BMad skill rendering runtime#127
chuanxu742-glitch merged 1 commit into
2233admin:mainfrom
chuanxu742-glitch:chuanxu742-glitch/bmad-runtime-restoration

Conversation

@chuanxu742-glitch

@chuanxu742-glitch chuanxu742-glitch commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

Restore the rendering runtime required by the repository's already-installed BMad skills, independently of #126 (browser/release contract tests).

  • Restore _bmad/scripts/render_skill.py, _bmad/scripts/config_utils.py, and the MIT license byte-for-byte from official bmad-code-org/BMAD-METHOD@beb368e5fc9b95bcec5e1de5bc7870dc15bece72 (skills/bmad/scripts/, license from repository root).
  • Add minimal project configuration (generic project paths and Chinese output settings, not recovered personal settings).
  • Ignore generated _bmad/render/ snapshots and document the local renderer invocation in TESTING.md.

This restores only these runtime files, not a complete BMad installation. No private configuration, generated snapshots, application runtime changes, or #126 release-contract changes are included. The two PRs modify distinct TESTING.md locations.

Verification

The renderer succeeded using absolute paths in an isolated checkout based on upstream 69c0ad1bca8ca4bfeb10ddd4541a17f831b38729; repository-relative equivalent:

uv run --no-cache _bmad/scripts/render_skill.py --project-root . --skill .agents/skills/bmad-build
  • Read the generated workflow and verified all 13 generated output hashes against its manifest; the snapshot was ignored by Git.
  • Compared both scripts and LICENSE against raw official files at the pinned revision: exact bytes match.
  • Confirmed the published branch contains exactly the six approved restoration files.

SHA-256 provenance:

  • render_skill.py: 8496d0d8b449d64c21b42a9aab3b13fc8a813a430c0695ffd84ad75bb1da7942
  • config_utils.py: 63e07c87f9290e1f3d73a97d88e5cf9be64152c80eb0dbb003ebe0be0864aed9
  • LICENSE: 0aa79baf6328b4a1e694ce10a12ffc36d7666554da128dff0e8fcda0fc536a66

This verifies skill rendering, not Docker image builds, container deployment, or release/tag publication.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 4da52a86-1fd8-4394-be12-e53e3a15a1c2

📥 Commits

Reviewing files that changed from the base of the PR and between 69c0ad1 and 4669c8a.

📒 Files selected for processing (6)
  • .gitignore
  • TESTING.md
  • _bmad/config.toml
  • _bmad/scripts/LICENSE
  • _bmad/scripts/config_utils.py
  • _bmad/scripts/render_skill.py

Warning

Your free Security trial is over. An organization admin can upgrade to Advanced for continuous pull request security review or dismiss this notice.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chuanxu742-glitch
chuanxu742-glitch merged commit da09f06 into 2233admin:main Sep 6, 2026
9 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