prr-kit 1.3.0 → 1.4.0

package/src/prr/workflows/3-review/business-review/workflow.yaml

@@ -7,11 +7,12 @@ user_name: "{config_source}:user_name"
 communication_language: "{config_source}:communication_language"
 target_repo: "{config_source}:target_repo"
 review_output: "{config_source}:review_output"
+session_output: "working-context:session_output || latest-session-folder-in:{review_output}"
 date: system-generated
 
 installed_path: "{project-root}/_prr/prr/workflows/3-review/business-review"
 instructions: "{installed_path}/instructions.xml"
 validation: "{installed_path}/checklist.md"
 
-pr_context: "{review_output}/current-pr-context.yaml"
-output_file: "{review_output}/business-review-{date}.md"
+pr_context: "working-context:target_branch,base_branch,pr_number,pr_knowledge_base"
+output_file: "{session_output}/business-review.md"

package/src/prr/workflows/3-review/general-review/checklist.md

@@ -22,6 +22,5 @@ validation-target: "General review output file"
 - [ ] Side effect findings include: the affected location OUTSIDE the diff (not just the changed file)
 
 ## Output
-- [ ] Findings written to `{review_output}/general-review-{date}.md`
-- [ ] PR context updated with `general-review` in completed list
+- [ ] Findings written to `{session_output}/general-review.md`
 - [ ] At least one positive observation included (balanced review)

package/src/prr/workflows/3-review/general-review/workflow.yaml

@@ -7,11 +7,12 @@ user_name: "{config_source}:user_name"
 communication_language: "{config_source}:communication_language"
 target_repo: "{config_source}:target_repo"
 review_output: "{config_source}:review_output"
+session_output: "working-context:session_output || latest-session-folder-in:{review_output}"
 date: system-generated
 
 installed_path: "{project-root}/_prr/prr/workflows/3-review/general-review"
 instructions: "{installed_path}/instructions.xml"
 validation: "{installed_path}/checklist.md"
 
-pr_context: "{review_output}/current-pr-context.yaml"
-output_file: "{review_output}/general-review-{date}.md"
+pr_context: "working-context:target_branch,base_branch,pr_number,pr_knowledge_base"
+output_file: "{session_output}/general-review.md"

package/src/prr/workflows/3-review/performance-review/checklist.md

@@ -20,5 +20,4 @@ validation-target: "Performance review output file"
 - [ ] ❓ QUESTION findings include: specific concern + context needed to assess severity (e.g., "Is this in a hot path?", "What is the expected data volume?")
 
 ## Output
-- [ ] Findings written to `{review_output}/performance-review-{date}.md`
-- [ ] PR context updated with `performance-review` in completed list
+- [ ] Findings written to `{session_output}/performance-review.md`

package/src/prr/workflows/3-review/performance-review/workflow.yaml

@@ -7,11 +7,12 @@ user_name: "{config_source}:user_name"
 communication_language: "{config_source}:communication_language"
 target_repo: "{config_source}:target_repo"
 review_output: "{config_source}:review_output"
+session_output: "working-context:session_output || latest-session-folder-in:{review_output}"
 date: system-generated
 
 installed_path: "{project-root}/_prr/prr/workflows/3-review/performance-review"
 instructions: "{installed_path}/instructions.xml"
 validation: "{installed_path}/checklist.md"
 
-pr_context: "{review_output}/current-pr-context.yaml"
-output_file: "{review_output}/performance-review-{date}.md"
+pr_context: "working-context:target_branch,base_branch,pr_number,pr_knowledge_base"
+output_file: "{session_output}/performance-review.md"

package/src/prr/workflows/3-review/security-review/checklist.md

@@ -22,5 +22,4 @@ validation-target: "Security review output file"
 - [ ] ❓ QUESTION findings include: specific concern + exact question to ask author (e.g., "Was this auth check intentionally removed?")
 
 ## Output
-- [ ] Findings written to `{review_output}/security-review-{date}.md`
-- [ ] PR context updated with `security-review` in completed list
+- [ ] Findings written to `{session_output}/security-review.md`

package/src/prr/workflows/3-review/security-review/workflow.yaml

@@ -7,6 +7,7 @@ user_name: "{config_source}:user_name"
 communication_language: "{config_source}:communication_language"
 target_repo: "{config_source}:target_repo"
 review_output: "{config_source}:review_output"
+session_output: "working-context:session_output || latest-session-folder-in:{review_output}"
 date: system-generated
 
 installed_path: "{project-root}/_prr/prr/workflows/3-review/security-review"
@@ -14,5 +15,5 @@ instructions: "{installed_path}/instructions.xml"
 validation: "{installed_path}/checklist.md"
 owasp_data: "{installed_path}/data/owasp-checklist.csv"
 
-pr_context: "{review_output}/current-pr-context.yaml"
-output_file: "{review_output}/security-review-{date}.md"
+pr_context: "working-context:target_branch,base_branch,pr_number,pr_knowledge_base"
+output_file: "{session_output}/security-review.md"

package/src/prr/workflows/4-improve/improve-code/checklist.md

@@ -13,6 +13,5 @@ validation-target: "Improve code output file"
 - [ ] Suggestions are correct — the AFTER code actually works
 
 ## Output
-- [ ] All suggestions written to `{review_output}/improve-code-{date}.md`
+- [ ] All suggestions written to `{session_output}/improve-code.md`
 - [ ] Suggestions grouped by category (Bugs | Quality | Performance | Best Practices)
-- [ ] PR context updated with `improve-code` in completed list

package/src/prr/workflows/4-improve/improve-code/workflow.yaml

@@ -7,11 +7,12 @@ user_name: "{config_source}:user_name"
 communication_language: "{config_source}:communication_language"
 target_repo: "{config_source}:target_repo"
 review_output: "{config_source}:review_output"
+session_output: "working-context:session_output || latest-session-folder-in:{review_output}"
 date: system-generated
 
 installed_path: "{project-root}/_prr/prr/workflows/4-improve/improve-code"
 instructions: "{installed_path}/instructions.xml"
 validation: "{installed_path}/checklist.md"
 
-pr_context: "{review_output}/current-pr-context.yaml"
-output_file: "{review_output}/improve-code-{date}.md"
+pr_context: "working-context:target_branch,base_branch,pr_number,pr_knowledge_base"
+output_file: "{session_output}/improve-code.md"

package/src/prr/workflows/5-ask/ask-code/steps/step-01-load-context.md

@@ -10,9 +10,9 @@ nextStepFile: "./step-02-answer.md"
 
 ### 1. Load PR Context
 
-Read `{review_output}/current-pr-context.yaml`.
+Use `session_output`, `target_branch`, `base_branch` from working context.
 
-If not found, prompt user to run [SP] Select PR first.
+If not set, prompt user to run [SS] Select Session or [SP] Select PR first.
 
 ### 2. Load the Diff
 

package/src/prr/workflows/6-report/generate-report/steps/step-01-collect.md

@@ -8,19 +8,23 @@ nextStepFile: "./step-02-organize.md"
 
 ## Sequence of Instructions
 
-### 1. Load PR Context
+### 1. Detect Completed Reviews
 
-Read `{review_output}/current-pr-context.yaml` to get list of completed reviews.
+Check which review files exist in `{session_output}/`:
+
+```bash
+ls "{session_output}/"
+```
 
 ### 2. Collect Review Output Files
 
-For each completed review in `review.completed` list, read the corresponding output file:
-- `general-review` → `{review_output}/general-review-*.md` (latest)
-- `security-review` → `{review_output}/security-review-*.md` (latest)
-- `performance-review` → `{review_output}/performance-review-*.md` (latest)
-- `architecture-review` → `{review_output}/architecture-review-*.md` (latest)
-- `business-review` → `{review_output}/business-review-*.md` (latest)
-- `improve-code` → `{review_output}/improve-code-*.md` (latest)
+For each review file that exists, read it:
+- `general-review` → `{session_output}/general-review.md`
+- `security-review` → `{session_output}/security-review.md`
+- `performance-review` → `{session_output}/performance-review.md`
+- `architecture-review` → `{session_output}/architecture-review.md`
+- `business-review` → `{session_output}/business-review.md`
+- `improve-code` → `{session_output}/improve-code.md`
 
 ### 3. Parse All Findings
 

package/src/prr/workflows/6-report/generate-report/steps/step-03-write.md

@@ -1,7 +1,7 @@
 ---
 name: "step-03-write"
 description: "Write the final review report to file"
-outputFile: "{review_output}/review-{target_branch_slug}-{date}.md"
+outputFile: "{session_output}/final-review.md"
 templateFile: "../templates/review-report.template.md"
 ---
 
@@ -27,7 +27,7 @@ The report includes:
 ```
 ✅ Review Report Generated!
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-📄 File: {review_output}/review-{target_branch_slug}-{date}.md
+📄 File: {session_output}/final-review.md
 
 Summary:
   🔴 Blockers: {blocker_count}

package/src/prr/workflows/6-report/generate-report/workflow.md

@@ -19,7 +19,8 @@ nextStep: "./steps/step-01-collect.md"
 ## INITIALIZATION
 
 Load config from `{main_config}`.
-Load PR context from `{review_output}/current-pr-context.yaml`.
+Use `session_output` from working context.
+If not set: prompt user to run [SS] Select Session or [SP] Select PR first.
 
 ## EXECUTION
 

package/src/prr/workflows/6-report/post-comments/steps/step-01-format.md

@@ -10,9 +10,9 @@ nextStepFile: "./step-02-post.md"
 
 ### 1. Check Prerequisites
 
-Verify `{review_output}/current-pr-context.yaml` exists and contains:
-- `pr_number` — if missing, show: `❌ No PR number found. Run [SP] Select PR first.`
-- `target_branch`, `base_branch`
+Verify working context contains:
+- `pr_number` — if missing, show: `❌ No PR number found. Run [SS] Select Session or [SP] Select PR first.`
+- `target_branch`, `base_branch`, `session_output`
 
 Verify platform CLI is available:
 
@@ -54,8 +54,7 @@ Store as `{commit_sha}`.
 
 ### 3. Load Review Report
 
-Find the latest report in `{review_output}/review-*.md`.
-Sort by modification time, take the most recent.
+Read the report at `{session_output}/final-review.md`.
 
 ### 4. Parse All Findings
 

package/src/prr/workflows/6-report/post-comments/workflow.md

@@ -37,10 +37,11 @@ Depends on `{active_platform}`:
 ## INITIALIZATION
 
 Load config from `{main_config}`.
-Load PR context from `{review_output}/current-pr-context.yaml`.
+Use `session_output`, `target_branch`, `base_branch`, `pr_number`, `active_platform` from working context.
+If `session_output` not set: prompt user to run [SS] Select Session or [SP] Select PR first.
 Use `{active_platform}` (or detect from `{platform}` config).
 
-Set `{temp_dir}` = `{review_output}/temp`.
+Set `{temp_dir}` = `{session_output}/temp`.
 Create it if it doesn't exist:
 ```bash
 mkdir -p "{temp_dir}"

package/src/prr/workflows/quick/workflow.md

@@ -152,26 +152,78 @@ git -C {target_repo} diff {base_branch}...{target_branch}
 ```
 Store diff in memory. Count files changed, lines added/removed.
 
-### 1e. Save PR context
-Write `{review_output}/current-pr-context.yaml`:
-```yaml
-pr:
-  target_branch: "{target_branch}"
-  base_branch: "{base_branch}"
-  pr_number: "{pr_number}"
-  pr_title: "{pr_title}"
-  platform: "{active_platform}"
-  platform_repo: "{platform_repo}"
-  date: "{date}"
-review:
-  completed: []
-  findings: []
+### 1e. Create Session Folder
+
+Compute the session output folder:
+
+```
+sanitized_branch = target_branch
+  .toLowerCase()
+  .replace(/[^a-z0-9]+/g, '-')
+  .replace(/^-+|-+$/g, '')
+  .substring(0, 40)
+
+if pr_number is set:
+  session_slug = "pr{pr_number}-{sanitized_branch}"
+else:
+  session_slug = "{sanitized_branch}"
+
+datetime_prefix = current datetime as "YYYY-MM-DD-HHmm"   (e.g. 2026-03-02-1430)
+
+session_output = "{review_output}/{datetime_prefix}-{session_slug}"
+```
+
+Examples:
+- `_prr-output/reviews/2026-03-02-1430-pr44-feature-auth-login`
+- `_prr-output/reviews/2026-03-02-1430-feature-auth-login`
+
+Create folder:
+```bash
+mkdir -p "{session_output}"
+```
+
+**Store in working context:** `session_output`, `target_branch`, `base_branch`, `pr_number`, `pr_title`, `active_platform`, `platform_repo`.
+
+### 1f. Generate Diffs Folder
+
+Parse the diff loaded in 1d and write per-file markdown files under `{session_output}/diffs/`, mirroring the repo folder tree.
+
+**For each changed file in the diff:**
+
+1. Determine file path, status, and line counts from the diff header.
+2. Construct output path: `{session_output}/diffs/{file_path}.md`
+3. Create parent directories:
+   ```bash
+   mkdir -p "{session_output}/diffs/{file_dir}"
+   ```
+4. Write the markdown file:
+
+```markdown
+# {file_path}
+**Status:** {modified|added|deleted|renamed} | **+{lines_added} / -{lines_removed} lines**
+---
+
+**Line {start_line}** *(or **Lines {start}–{end}** for multi-line hunks)*
+```diff
+- old line content
++ new line content
 ```
 
-Show summary:
+*(repeat for each hunk in this file)*
+```
+
+**Rules:**
+- **New file:** show all lines as `+` additions under a single hunk block.
+- **Deleted file:** show all lines as `-` deletions under a single hunk block.
+- **Renamed file with no content change:** write status line only, no diff blocks.
+- **Renamed file with changes:** write status `renamed` and include change hunks as normal.
+- One `.md` file per changed file. Folder structure mirrors the repo exactly.
+
+After writing all files, show summary:
 ```
 ✓ PR selected: {target_branch}
   Files changed: X | +Y / -Z lines
+✓ Diffs saved: {file_count} files → {session_output}/diffs/
 ```
 
 ---
@@ -202,12 +254,12 @@ Print to screen:
 ---
 
 ## PHASE 2.5 — COLLECT PR-SPECIFIC CONTEXT
-*Execute automatically. May pause once to ask the user for additional context (unless `skip_manual_input_context: true` in config).*
+*Execute automatically. Pauses once to ask the user for additional context — user must respond before continuing.*
 
 Execute the collect-pr-context workflow in full:
 `{project-root}/_prr/prr/workflows/2-analyze/collect-pr-context/workflow.md`
 
-This workflow analyzes changed files, detects technology stacks, collects relevant context from all sources (primary docs, config files, standards docs, inline annotations, stack-specific rules, external MCP/RAG tools), asks the user for any additional context (marked ⚠️ IMPORTANT if provided), and builds a structured PR-specific knowledge base at `{review_output}/pr-{pr_number}-context.yaml`.
+This workflow analyzes changed files, detects technology stacks, collects relevant context from all sources (primary docs, config files, standards docs, inline annotations, stack-specific rules, external MCP/RAG tools), asks the user for any additional context (marked ⚠️ IMPORTANT if provided), and builds a structured PR-specific knowledge base at `{session_output}/pr-context.yaml`.
 
 On completion, store `pr_knowledge_base` = path to the generated context file.
 
@@ -217,7 +269,7 @@ On completion, store `pr_knowledge_base` = path to the generated context file.
 *Execute all review types automatically, one by one.*
 
 **Before each review, set these variables so the review instructions resolve correctly:**
-- `pr_context` = `{review_output}/current-pr-context.yaml`
+- `pr_context` = working context (`target_branch`, `base_branch`, `pr_number`, `pr_knowledge_base`)
 - `target_branch` = `pr_context.pr.target_branch`
 - `base_branch` = `pr_context.pr.base_branch`
 - `pr_number` = `pr_context.pr.pr_number`
@@ -225,36 +277,44 @@ On completion, store `pr_knowledge_base` = path to the generated context file.
 - `target_repo`, `communication_language` = from config
 - `output_file` = per-review path defined below *(ensures findings are saved to disk for [RR] and [PC] later)*
 
+**Scope gate:** Read `user_instructions.review_scope` from `{pr_knowledge_base}`.
+- If `"all"` (or knowledge base missing) → run all reviews 3a–3e normally.
+- If a list (e.g. `[SR, AR]`) → only run reviews whose code is in the list. For each skipped review, print:
+  `⏭️ {Review Name} skipped (not in scope)`
+  and do NOT write its output file.
+
+Review codes: `GR` = General · `SR` = Security · `PR` = Performance · `AR` = Architecture · `BR` = Business
+
 ### 3a. General Review
-Set `output_file` = `{review_output}/general-review-{date}.md`
+Set `output_file` = `{session_output}/general-review.md`
 Load and follow: `{project-root}/_prr/prr/workflows/3-review/general-review/instructions.xml`
 
 Collect findings as `{general_findings}`.
 Print section header: `## 👁️ General Review`
 
 ### 3b. Security Review
-Set `output_file` = `{review_output}/security-review-{date}.md`
+Set `output_file` = `{session_output}/security-review.md`
 Load and follow: `{project-root}/_prr/prr/workflows/3-review/security-review/instructions.xml`
 
 Collect findings as `{security_findings}`.
 Print section header: `## 🔒 Security Review`
 
 ### 3c. Performance Review
-Set `output_file` = `{review_output}/performance-review-{date}.md`
+Set `output_file` = `{session_output}/performance-review.md`
 Load and follow: `{project-root}/_prr/prr/workflows/3-review/performance-review/instructions.xml`
 
 Collect findings as `{performance_findings}`.
 Print section header: `## ⚡ Performance Review`
 
 ### 3d. Architecture Review
-Set `output_file` = `{review_output}/architecture-review-{date}.md`
+Set `output_file` = `{session_output}/architecture-review.md`
 Load and follow: `{project-root}/_prr/prr/workflows/3-review/architecture-review/instructions.xml`
 
 Collect findings as `{architecture_findings}`.
 Print section header: `## 🏗️ Architecture Review`
 
 ### 3e. Business Review
-Set `output_file` = `{review_output}/business-review-{date}.md`
+Set `output_file` = `{session_output}/business-review.md`
 Load and follow: `{project-root}/_prr/prr/workflows/3-review/business-review/instructions.xml`
 
 Collect findings as `{business_findings}`.
@@ -276,10 +336,7 @@ Count totals:
 - `{warning_count}` = number of 🟡 findings
 - `{suggestion_count}` = number of 🟢 findings
 
-Generate report filename: `review-{target_branch_slug}-{date}.md`
-where `{target_branch_slug}` = branch name with `/` replaced by `-`.
-
-Write report to: `{review_output}/review-{target_branch_slug}-{date}.md`
+Write report to: `{session_output}/final-review.md`
 
 Report format:
 ```markdown
@@ -328,7 +385,8 @@ Print completion summary:
 ✅ Quick Review Complete
 
 Branch:  {target_branch}
-Report:  {review_output}/review-{target_branch_slug}-{date}.md
+Session: {session_output}/
+Report:  {session_output}/final-review.md
 
 🔴 Blockers:    {blocker_count}
 🟡 Warnings:    {warning_count}