diff --git a/AI/commands/srid-do.md b/AI/commands/srid-do.md index e59b115..c59acf5 100644 --- a/AI/commands/srid-do.md +++ b/AI/commands/srid-do.md @@ -35,7 +35,7 @@ Take a GitHub issue, prompt, or task description and execute it top-to-bottom: i ### 3. **Open Draft PR** - Push the branch and open a **draft** pull request using `gh pr create --draft`. - - **Use the `github-pr` skill** for the title and body — no bullet-list dumps. + - **MANDATORY**: Load the `github-pr` skill (via `Skill` tool) BEFORE writing the PR title/body. Follow it exactly — paragraph-based descriptions, no bullet-list dumps. ### 4. **CI Loop** diff --git a/AI/memory.md b/AI/memory.md index 106c60e..5317060 100644 --- a/AI/memory.md +++ b/AI/memory.md @@ -14,3 +14,4 @@ - **nix tools**: If any tool is unavailable, get it from nixpkgs: `nix run nixpkgs#`. - **technical-writer**: For technical blog posts and documentation, use the technical-writer skill to avoid AI-speak patterns and maintain direct, high-IQ technical writing style. +- **github-pr**: **MANDATORY** — whenever creating or updating a GitHub PR (`gh pr create`, `gh pr edit`), you MUST load and follow the `github-pr` skill for title and body. No exceptions. No bullet-list dumps. Read the skill first, then write the PR. diff --git a/AI/skills/github-pr/SKILL.md b/AI/skills/github-pr/SKILL.md index dcecdf8..5b88228 100644 --- a/AI/skills/github-pr/SKILL.md +++ b/AI/skills/github-pr/SKILL.md @@ -38,6 +38,17 @@ Write PR descriptions that fellow devs actually want to read. - Link to issues/discussions where relevant (`Closes #123`, `See #456`) - If the PR is trivial (typo fix, version bump), a one-liner body is fine +## Try it locally + +If the repo is a Nix flake and the PR branch contains a buildable output (package, NixOS config, etc.), include a "Try it locally" section at the end of the body. Use the GitHub owner/repo and branch name to construct the command: + +``` +### Try it locally +`nix run github://` +``` + +Adjust the command as needed — `nix build` for non-runnable outputs, add `#` if the default package isn't the relevant one. Omit this section entirely if the change isn't meaningfully testable via `nix run/build` (e.g., CI-only changes, documentation, non-Nix repos). + ## Updating existing PRs When the user pushes further changes to an already-PR'd branch: diff --git a/flake.lock b/flake.lock index a073f74..111f617 100644 --- a/flake.lock +++ b/flake.lock @@ -862,11 +862,11 @@ "systems": "systems_3" }, "locked": { - "lastModified": 1774230611, - "narHash": "sha256-cOiYBVZIGzCE6r9NM4F8vyMptKahKL3+LNb1RIoCj3U=", + "lastModified": 1774271165, + "narHash": "sha256-55d2JYsay/SnAZcA2eOT6H/miLa685+A/A2lYf9MxFI=", "owner": "juspay", "repo": "kolu", - "rev": "86689beb1498f0ba432d059bf86f58adb97d5aaf", + "rev": "f30acd88f63c4edb77b17c665bb8a5a8d4a91bdd", "type": "github" }, "original": {