pull_request_templates 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e611a2cbdb56bda1b85bc97bcd1e9ba850944a956aca3772e2479e2080780ed0
-  data.tar.gz: f22afc24c2cab63778da2174ffdbde15a375a0f6d96b8643ffe7e5ec9a10dfd5
+  metadata.gz: 95ddd43a03464906a1436dab7a0167817113bf43fe5d8c59c53fb7b8658da2f1
+  data.tar.gz: 7f385ff70f9be681072eaa3c5b52e6290a8edc6c902dc2ae41ecaae7201fdcce
 SHA512:
-  metadata.gz: e285e78b50d6a53f31a3bf8c1c8162816e6310175a1ddda470d01490e82ff1744d6663a392870992adead8bd2856a609024e62e93218629aa3b2bfe925a9beee
-  data.tar.gz: a0c10b8ee580963bd2a109f4d262a311681d76558e486ae0ecfa2f08c007134a6ef99b9762deff4ab0fb3edb7398843e7e0a3f2c98b6dc88d60e18a57db593b8
+  metadata.gz: 6b914f1efe7ee44d378c53a2468de2283914d7e4a125f4079bb7408963a633e2dc8284544bff47a6b9b26921b9b5f951f0b64b4ba8ebd89182a17a2c60445228
+  data.tar.gz: d60ea637488d20978570ecefaee4c4da01ea6fcdc7e91c3b2b6ca2a1088d0715617e3deb9d96fdef6a35177b811f1dbacba78ea5c5a2e74e43f8cbd6ff0dc844

data/.cursor/rules/test-driven-developer.mdc

@@ -0,0 +1,59 @@
+---
+description: 
+globs: *_spec.rb
+alwaysApply: false
+---
+# TDD Workflow Rule
+
+## Core Priorities
+1. Always write a failing test before implementing behavior
+2. Automatically implement changes when test failures match expectations
+
+## Workflow Steps
+
+1. **Write Test**
+   - Write a minimal test to demonstrate missing behavior
+   - Include specific error message expectations
+   - Follow RSpec best practices
+
+2. **Verify Test Failure**
+   - Run the specific failing test
+   - If the failure matches expectations, proceed to implementation
+   - If the failure is unexpected, revise the test
+
+3. **Commit the Failing Test**
+   - Commit the failing test separately from the implementation
+   - Use conventional commits with gitmoji
+   - Format: `🧪 test(scope): description` for tests
+   - Include a `Co-authored-by:` trailer if AI contributed
+
+4. **Implement Behavior**
+   - Write minimal code to pass the test
+   - Follow Ruby best practices
+   - Maintain existing code style
+
+5. **Commit the Implementation**
+   - Commit the implementation separately from the test
+   - Use conventional commits with gitmoji
+   - Format: `✨ feat(scope): description` for implementation
+   - Include a `Co-authored-by:` trailer if AI contributed
+
+6. **Review Confidence**
+   - The goal is to build confidence in the process so that you can review the changes in a PR rather than accepting every change in the agent session
+   - If you get stuck, you are encouraged to ask for help, but strict adherence to the process is required
+
+## Gitmoji Reference
+- 🧪 for failing test commits
+- ✨ for feature implementations
+- ♻️ for refactoring
+- 🐛 for bug fixes
+- 🔧 for configuration changes
+- 🚚 for renaming or moving
+- 🏗️ for architectural changes
+
+## Notes
+- Keep tests focused and atomic
+- One behavior per test
+- Maintain test isolation
+- Follow red-green-refactor cycle strictly
+- Always attribute AI contributions in commit messages

data/lib/pull_request_templates/cli.rb

@@ -52,30 +52,63 @@ module PullRequestTemplates
       changes.split("\n").reject(&:empty?)
     end
 
-    def select_template(templates, changes)
-      mapping_file = ".github/PULL_REQUEST_TEMPLATE/.mapping.yml"
+    def select_template(template_files, changes)
+      mapping_file = ".github/PULL_REQUEST_TEMPLATE/config.yml"
       candidates = []
       if File.exist?(mapping_file)
-        mapping = YAML.load_file(mapping_file)
+        templates = YAML.load_file(mapping_file).fetch("templates")
         matches = Hash.new { |h, k| h[k] = [] }
         changes.each do |file|
-          mapping.each do |template, patterns|
+          templates.each do |template|
+            patterns = template.fetch("pattern")
             Array(patterns).each do |pattern|
-              matches[template] << file if File.fnmatch(pattern, file, File::FNM_PATHNAME | File::FNM_EXTGLOB)
+              matches[template] << file if File.fnmatch(pattern, file, File::FNM_PATHNAME | File::FNM_EXTGLOB | File::Constants::FNM_DOTMATCH)
             end
           end
         end
         selected = matches.select { |_, files| files.sort == changes.sort }
         candidates = selected.keys if selected.any?
       end
-      candidates = templates if candidates.empty?
+      candidates = template_files.map { {"file" => _1} } if candidates.empty?
+
+      # If we have a template with fallback: true, use it when multiple templates match
+      if candidates.length > 1
+        fallback = candidates.find { _1.fetch("fallback", false) }
+        return fallback.fetch("file") if fallback
+      end
+
       if candidates.length == 1
-        return candidates.first
+        return candidates.first.fetch("file")
+      end
+
+      if File.exist?(mapping_file)
+        raise AmbiguousTemplateSelection, <<~MESSAGE
+          Unable to pick one template from #{candidates.map { _1.fetch("file") }} for the changes to #{changes.count} files:
+          * #{changes.join("\n* ")}
+
+          To resolve this, add a fallback template to your config.yml:
+          - file: default.md
+            pattern: "**/*"
+            fallback: true
+        MESSAGE
+      else
+        raise AmbiguousTemplateSelection, <<~MESSAGE
+          Unable to pick one template from #{candidates.map { _1.fetch("file") }} for the changes to #{changes.count} files:
+          * #{changes.join("\n* ")}
+
+          To resolve this, add a fallback template to your config.yml:
+          - file: default.md
+            pattern: "**/*"
+            fallback: true
+
+          Run this command to create the fallback template:
+          echo 'templates:
+            - file: default.md
+              pattern: "**/*"
+              fallback: true
+          ' >> .github/PULL_REQUEST_TEMPLATE/config.yml
+        MESSAGE
       end
-      raise AmbiguousTemplateSelection, <<~MESSAGE
-        Unable to pick one template from #{candidates} for the changes to #{changes.count} files:
-        * #{changes.join("\n* ")}
-      MESSAGE
     end
 
     def generate_pr_url(branch, template)

data/lib/pull_request_templates/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PullRequestTemplates
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/lib/pull_request_templates.rb

@@ -3,7 +3,10 @@
 require_relative "pull_request_templates/version"
 require_relative "pull_request_templates/cli"
 
+require "thor/error"
+
 module PullRequestTemplates
   Error = Class.new(StandardError)
-  AmbiguousTemplateSelection = Class.new(Error)
+  CliError = Class.new(Thor::Error)
+  AmbiguousTemplateSelection = Class.new(CliError)
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pull_request_templates
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Caleb Buxton
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-05-11 00:00:00.000000000 Z
+date: 2025-06-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -33,6 +33,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".cursor/rules/test-driven-developer.mdc"
 - ".rspec"
 - ".standard.yml"
 - CHANGELOG.md