Simplify PR template metadata requirements

Author: QuantumMisaka

.github/pull_request_template.md

@@ -1,51 +1,15 @@
-### Reminder
-- [ ] I have read `AGENTS.md` and `docs/developers_guide/agent_governance.md`.
-- [ ] I have linked an issue or explained why this PR does not need one.
-- [ ] I have added adequate unit tests and/or case tests, or explained why not.
-- [ ] I have listed the exact verification commands run and their results.
-- [ ] I have described user-visible behavior changes, including INPUT parameter changes.
-- [ ] I have explained core-module impact for ESolver, HSolver, ElecState, Hamilt, Operator, Psi, or other `source/` changes.
-- [ ] I have requested any needed governance exception below.
-
 ### Linked Issue
-Fix #...
+Fix #
 
 ### Unit Tests and/or Case Tests for my changes
-- A unit test is added for each new feature or bug fix.
-
-### Exact Verification Performed
 - Commands run:
 - Result summary:
 - Checks not run, with reason:
 
 ### What's changed?
-- Example: My changes might affect the performance of the application under certain conditions, and I have tested the impact on various scenarios...
-
-### Governance Checklist
-- Global dependencies: no net increase in `GlobalV`, `GlobalC`, or `PARAM` code references, or exception requested below with reason, scope, risk, and cleanup plan.
-- Default parameters: no new default arguments added to existing interfaces, or exception requested below.
-- Headers: no unnecessary header dependencies or `.hpp` propagation, or rationale provided below.
-- Line endings: text files use LF; only `.bat` and `.cmd` use CRLF.
-- Build linkage: new source files are listed in the relevant `CMakeLists.txt`, or rationale provided below.
-- Documentation: behavior/interface changes include documentation updates, or no documentation update is required because ...
-- CodeRabbit: if automatic review has not started and the repository has CodeRabbit installed, request `@coderabbitai review`.
-
-### INPUT Parameter Changes
-- Parameters added/removed/changed:
-- `docs/parameters.yaml` updated: yes/no/not applicable
-- `docs/advanced/input_files/input-main.md` updated: yes/no/not applicable
-- If not updated, explain why no INPUT documentation update is required:
-
-### Core Module Impact
-- Affected core modules:
-- Risk summary:
-- Compatibility or performance impact:
+- Example: brief summary of the user-visible or developer-facing change.
 
-### Governance Exception
-- Rule:
-- Reason:
-- Scope:
-- User or maintenance risk:
-- Why the normal rule cannot be followed now:
-- Follow-up cleanup plan:
-- Requested approver:
+### Governance Notes
+- INPUT/docs changes:
+- Core module impact:
+- Exceptions requested:

tools/03_code_analysis/agent_governance_check.py

@@ -542,10 +542,6 @@ def check_pr_metadata(findings: List[Finding], body: Optional[str]) -> None:
         "Linked Issue",
         "Unit Tests and/or Case Tests for my changes",
         "What's changed?",
-        "Governance Checklist",
-        "INPUT Parameter Changes",
-        "Core Module Impact",
-        "Governance Exception",
     ]
     sections = pr_sections(body)
     missing = [section for section in required_sections if section not in sections]
@@ -567,7 +563,7 @@ def check_pr_metadata(findings: List[Finding], body: Optional[str]) -> None:
             "pull_request.body",
             None,
             "; ".join(reason_parts),
-            "Fill the PR template with issue linkage, test evidence, behavior impact, governance notes, and exception details.",
+            "Fill the PR template with issue linkage, test evidence, and a concise change summary.",
         )
 
 

tools/03_code_analysis/test_agent_governance_check.py

@@ -322,7 +322,7 @@ def test_skips_pr_metadata_when_event_payload_is_not_a_pull_request(self):
         self.assertEqual(result.returncode, 0, result.stdout + result.stderr)
         self.assertNotIn("PR metadata completeness", result.stdout)
 
-    def test_accepts_filled_pr_template_fields_from_event_payload(self):
+    def test_accepts_core_pr_template_fields_from_event_payload(self):
         event = self.repo / "event.json"
         event.write_text(
             json.dumps(
@@ -332,15 +332,7 @@ def test_accepts_filled_pr_template_fields_from_event_payload(self):
                         "### Unit Tests and/or Case Tests for my changes\n"
                         "Ran python3 -m unittest tools/03_code_analysis/test_agent_governance_check.py.\n\n"
                         "### What's changed?\n"
-                        "Adds governance checks only; no runtime behavior change.\n\n"
-                        "### Governance Checklist\n"
-                        "Line endings, CMake linkage, and docs rules reviewed.\n\n"
-                        "### INPUT Parameter Changes\n"
-                        "No INPUT parameter changes.\n\n"
-                        "### Core Module Impact\n"
-                        "No core module impact.\n\n"
-                        "### Governance Exception\n"
-                        "No exceptions requested.\n"
+                        "Adds governance checks only; no runtime behavior change.\n"
                     }
                 }
             )
@@ -349,6 +341,7 @@ def test_accepts_filled_pr_template_fields_from_event_payload(self):
         result = self.run_checker("--event-path", str(event))
 
         self.assertEqual(result.returncode, 0, result.stdout + result.stderr)
+        self.assertNotIn("PR metadata completeness", result.stdout)
 
     def test_warns_for_source_change_without_test_evidence(self):
         self.write("source/source_base/new_feature.cpp", "int new_feature() { return 1; }\n")