Skip to content

Drop the per instance matches <schema> line from validate - #854

Merged
jviotti merged 1 commit into
mainfrom
matches-validat
Sep 7, 2026
Merged

jviotti merged 1 commit into
mainfrom
matches-validat

Conversation

@jviotti

@jviotti jviotti commented Sep 7, 2026 •

Copy link
Copy Markdown
Member

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Review in cubic

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@augmentcode

augmentcode Bot commented Sep 7, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR streamlines successful validate --verbose output.

  • Removes the per-instance matches <schema> line after successful validations.
  • Simplifies process_entry by dropping its now-unused schema display-path parameter.
  • Eliminates the corresponding schema display-path calculation in validate.
  • Applies the concise success format to both batched inputs and direct instance-file processing.
  • Updates CLI integration-test expectations across files, directories, JSONL, YAML, stdin, and resolver scenarios.
Technical Notes: Failure reporting, JSON output, tracing, benchmarking, and validation summaries remain unchanged.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 59 files

Re-trigger cubic

@jviotti
jviotti merged commit c8fa0b4 into main Sep 7, 2026
16 checks passed
@jviotti
jviotti deleted the matches-validat branch September 7, 2026 18:09
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