Skip to content

Add CLAUDE.md with architecture and contract notes (AST-148815)#19

Closed
cx-anurag-dalke wants to merge 2 commits into
mainfrom
docs/add-claude-md
Closed

Add CLAUDE.md with architecture and contract notes (AST-148815)#19
cx-anurag-dalke wants to merge 2 commits into
mainfrom
docs/add-claude-md

Conversation

@cx-anurag-dalke

@cx-anurag-dalke cx-anurag-dalke commented Apr 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Adds a CLAUDE.md at the repo root documenting the Parser interface / ParsersFactory dispatch model and how to wire a new ecosystem.
  • Captures per-parser quirks (Maven multi-line Locations, npm lockfile fallback, pypi ==-only scope, dotnet variants).
  • Documents invariants downstream AST-CLI depends on: 0-based line numbers, literal "latest" for unresolvable versions, stable PackageManager strings (mvn/npm/pypi/go/nuget).
  • Notes the 60% CI coverage floor.

Test plan

  • Reviewer confirms architecture section matches current code.
  • Reviewer confirms invariants (0-based lines, "latest" sentinel, PackageManager values) are accurate.
  • No code changes; CI should pass unchanged.

@cx-anurag-dalke @claude
Add CLAUDE.md with architecture and contract notes
bd0e3a8 
Captures the Parser/factory dispatch model, per-ecosystem quirks,
and the invariants (0-based line numbers, "latest" sentinel,
PackageManager strings) that downstream AST-CLI relies on.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Apr 20, 2026 •
edited
Loading

Copy link
Copy Markdown

Logo
Checkmarx One – Scan Summary & Details – e55a858e-4c7e-4cf4-a5ac-4e66b097ede0

Great job! No new security vulnerabilities introduced in this pull request

@cx-anurag-dalke cx-anurag-dalke changed the title Add CLAUDE.md with architecture and contract notes Add CLAUDE.md with architecture and contract notes (AST-148815) Apr 21, 2026
@cx-anurag-dalke cx-anurag-dalke deleted the docs/add-claude-md branch April 21, 2026 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cx-umesh-waghode cx-umesh-waghode Awaiting requested review from cx-umesh-waghode

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cx-anurag-dalke