Skip to content

validate: pass repo bundle package; adopt quire-rs 0.53.0 (PLAT-50) - #150

Merged
kreneskyp merged 1 commit into
mainfrom
plat-1012-1056-178-179-1177-50
Nov 8, 2026
Merged

kreneskyp merged 1 commit into
mainfrom
plat-1012-1056-178-179-1177-50

Conversation

@kreneskyp

Copy link
Copy Markdown
Contributor

Closes PLAT-50. Bumps quire-rs to v0.53.0 (rev ce172a7), adds the new ValidationReason::Regex remedy arm, and has quire validate derive <org>/<repo> from the scope's origin remote and call validate_document_in_bundle. No remote: falls back to the previous call.

Not included: PLAT-1012 (needs a spec cycle: FR-022/023/024, NFR-008, US-007, ADR-0002 mandate the launcher and quire-dist packaging).

Gate: fmt-check, clippy -D warnings, cargo test --workspace --no-fail-fast: all pass except it_144 (python jsonschema module missing on this host).

🤖 Generated with Claude Code

validate resolves <org>/<repo> from the scope's origin remote and calls
validate_document_in_bundle so relationship targets are checked instead
of falling to no-bundle-package.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@kreneskyp
kreneskyp requested a review from a team as a code owner November 8, 2026 05:16

@kreneskyp kreneskyp left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lead review (diff only): PLAT-50 validate takes / from the scope's origin remote (https/ssh/scp parse, unit-tested) and calls validate_document_in_bundle, falling back to the old call with no remote; Regex remedy arm for new ValidationReason::Regex; quire-rs bumped to =0.53.0 at ce172a7. Gate: fmt-check and lint green; tests green except it_144 (host: python probe lacks jsonschema module), not caused by this change. make ci (docs/deny/audit/spec) not run. Known gap: no test where a relationships row lowers under the repo package (no fixture module in quire-cli). No findings.

@kreneskyp
kreneskyp merged commit 94ced87 into main Nov 8, 2026
3 checks passed
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