Skip to content

fix: tighten request validation - #1780

Open
MantisClone wants to merge 1 commit into
masterfrom
fix/REQ-926-request-validation
Open

MantisClone wants to merge 1 commit into
masterfrom
fix/REQ-926-request-validation

Conversation

@MantisClone

Copy link
Copy Markdown
Contributor

Problem

Request validation needs an additional consistency check.

References REQ-926.

Proposed Solution

Tighten validation and add regression coverage.

Design Rationale

Keep the check in the existing request-processing path.

Considerations

Local package tests, type checking and formatting pass. Detailed context remains in the private issue.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@MantisClone
MantisClone marked this pull request as ready for review November 7, 2026 00:01
@greptile-apps

greptile-apps Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Adds a case-insensitive requestId check before applying updates.

  • Request updates only change the request named by their ID.

No actionable issues found. Tests were reviewed but not run.

Reviews (1) · Last reviewed commit: "fix: tighten request validation" · Reviewed by Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant