Skip to content

docs: agree on the golden-AST and robustness counts across the docs - #175

Merged
HuiJun merged 1 commit into
mainfrom
devin/1786780144-doc-counts
Aug 15, 2026
Merged

docs: agree on the golden-AST and robustness counts across the docs#175
HuiJun merged 1 commit into
mainfrom
devin/1786780144-doc-counts

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Aug 15, 2026

Copy link
Copy Markdown

Summary

#174 added one golden AST fixture (parse/behavior_first_then_declared_node) and three
robustness subtests, and updated the counts in README.md and docs/SPEC_COMPLIANCE.md
only — so docs/ARCHITECTURE.md, docs/TESTING.md, docs/ROADMAP.md and the
implementation-files table in docs/SPEC_COMPLIANCE.md still said 80 fixtures / 143
subtests, contradicting the same numbers a few lines away and the tree itself
(ls internal/core/parser/testdata/parse/*.golden = 81).

Docs-only: 80 → 81 golden ASTs, 143 → 146 robustness subtests, in the six places that
still carried the old values. docs/RELEASING.md requires these to agree across the docs
that repeat them.

Link to Devin session: https://nasa-jpl-demo.devinenterprise.com/sessions/63a35983480640b28891656068676bd8
Requested by: @HuiJun


Open in Devin Review

Co-Authored-By: jason.han <jason.han@jpl.nasa.gov>
@HuiJun HuiJun self-assigned this Aug 15, 2026
@devin-ai-integration

Copy link
Copy Markdown
Author
Original prompt from jason.han

@JPL-Devin/Systemica Help me find any gaps in useability in the features we've implemented.

@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@HuiJun
HuiJun merged commit 8d373d0 into main Aug 15, 2026
2 of 3 checks passed

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

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