Skip to content

test(structural): add parser failure golden fixture - #44

Closed
nia-sg-bot wants to merge 2 commits into
mainfrom
nia/issue-22-syntax-failure-fixture
Closed

test(structural): add parser failure golden fixture#44
nia-sg-bot wants to merge 2 commits into
mainfrom
nia/issue-22-syntax-failure-fixture

Conversation

@nia-sg-bot

Copy link
Copy Markdown
Contributor

Summary

  • add a checked-in golden artifact for a malformed Python snapshot
  • make the parser-failure regression compare the schema-valid, scoped warning artifact against that fixture
  • lock the deterministic no-topology outcome for syntax errors: no symbols or relationships, with an explicit PARSE_FAILURE warning

Why this slice

Issue #22 requires golden coverage for parser failures and guarantees that parse failures become scoped warnings rather than silent empty snapshots. This narrow fixture makes that behavior reviewable and protects the deterministic Tree-sitter baseline without changing the public schema or CLI contract.

Compatibility

  • no artifact schema, CLI behavior, or extension-facing contract changes
  • the fixture asserts the existing schema_version: 2.0 structural output subset and warning semantics

Validation

  • python3 -m pytest -q — 150 passed
  • python3 -m compileall -q diffgraph tests
  • git diff --check origin/main...HEAD
  • python3 -m diffgraph.cli --help

Part of #22

Remaining work

Issue #22 remains open for the broader deterministic Tree-sitter baseline and its remaining fixtures.

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: bf34c2ad-f0f0-434e-9c6c-b97dde31cd34


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@nia-sg-bot nia-sg-bot added roadmap Tracked on the public WildestAI roadmap priority:P0 Critical path / blocks a usable product direction:aligned Aligned with the current WildestAI product direction labels Sep 3, 2026
@nia-sg-bot

Copy link
Copy Markdown
Contributor Author

This PR appears to be a duplicate of already-merged PR #43 (test(structural): add parser failure golden fixture). The changes from PR #43 were merged to main on 2026-09-01T20:02:43Z by Avikalp. Closing as no further action needed.

@nia-sg-bot nia-sg-bot closed this Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

direction:aligned Aligned with the current WildestAI product direction priority:P0 Critical path / blocks a usable product roadmap Tracked on the public WildestAI roadmap

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant