[PR #3427] [CLOSED] fix(rea): inspection substeps for completion inventories #5736

New issue

Closed

opened 2026-10-31 01:09:03 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-31 01:09:03 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/3427
Author: @CRYPTONIKAV
Created: 10/14/2026
Status: ❌ Closed

Base: main ← Head: fix/3220-inspection-substeps

📝 Commits (2)

c45f674 fix(rea): offer every inspection substep inventory in completion
b505de5 test(rea): assert occurrences across substeps

📊 Changes

2 files changed (+63 additions, -10 deletions)

View changed files

📝 src/server/promptCompletion.ts (+19 -10)
📝 tests/boundary/mcp/promptCompletion.test.ts (+44 -0)

📄 Description

Fixes #3220

What I changed

Banked fix from earlier lane work (branch fix/3220-inspection-substeps): inspection substeps for prompt completion inventories.
Verified locally with the repo's fast checks.

Verification

npm run check:fast (typecheck + lint) green; focused tests green.

Author readiness

Human owner: @CRYPTONIKAV

I reviewed the final diff and relevant existing code, understand the change and its edge cases, and take responsibility for the complete PR, including AI-assisted work.
I checked the evidence above, confirmed it supports the current change, and recorded meaningful validation gaps.
I declare this PR ready for initial maintainer review.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/3427 **Author:** [@CRYPTONIKAV](https://github.com/CRYPTONIKAV) **Created:** 10/14/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/3220-inspection-substeps` --- ### 📝 Commits (2) - [`c45f674`](https://github.com/morluto/rea/commit/c45f674156b803fa1a10f26f17cafe99d85b2489) fix(rea): offer every inspection substep inventory in completion - [`b505de5`](https://github.com/morluto/rea/commit/b505de5c10791fdfc6c1c0c79393ab432603c7f6) test(rea): assert occurrences across substeps ### 📊 Changes **2 files changed** (+63 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `src/server/promptCompletion.ts` (+19 -10) 📝 `tests/boundary/mcp/promptCompletion.test.ts` (+44 -0) </details> ### 📄 Description Fixes #3220 ## What I changed - Banked fix from earlier lane work (branch `fix/3220-inspection-substeps`): inspection substeps for prompt completion inventories. - Verified locally with the repo's fast checks. ## Verification - `npm run check:fast` (typecheck + lint) green; focused tests green. ## Author readiness Human owner: @CRYPTONIKAV - [x] I reviewed the final diff and relevant existing code, understand the change and its edge cases, and take responsibility for the complete PR, including AI-assisted work. - [x] I checked the evidence above, confirmed it supports the current change, and recorded meaningful validation gaps. - [x] I declare this PR ready for initial maintainer review. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-31 01:09:03 +00:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

morluto/rea#5736

No description provided.

Rows
Columns