Skip to content

Open Forge PRs ready for review #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
GolaraJ merged 1 commit into from
Aug 20, 2026
Merged

Open Forge PRs ready for review #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions plugins/repo-dreamer/skills/repository-skill-forge/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -375,6 +375,9 @@ Set `$SELECTED_PROPOSAL_DIR` to the selected run-local skill directory and
`$SKILL_PATH` to its repository-relative destination before the checkout
validation steps.

For every `create` action, pass `draft: false` explicitly so newly created
Forge PRs open ready for review. Never omit the field or rely on the platform
default.
For a create action, stage only the selected proposal. Before copying, reject
symlinks in its source tree, every existing destination path component, and
every existing destination entry mapped from a selected source entry. Compare
Expand Down
8 changes: 8 additions & 0 deletions plugins/repo-dreamer/skills/repository-skill-forge/tests/test_stateless_forge.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -278,5 +278,13 @@ def test_selection_rejects_invalid_rank(self) -> None:
ledger.select([invalid], [])


class PublicationContractTests(unittest.TestCase):
def test_forge_opens_new_pull_requests_ready_for_review(self) -> None:
contract = (SKILL_DIR / "SKILL.md").read_text(encoding="utf-8")

self.assertIn("pass `draft: false` explicitly", contract)
self.assertIn("open ready for review", contract)


if __name__ == "__main__":
unittest.main()