Skip to content

docs: adds claude.md files#36

Merged
lucasloisp merged 2 commits into
mainfrom
llois-c/docs_adds_claude.md_files
Jul 2, 2026
Merged

docs: adds claude.md files#36
lucasloisp merged 2 commits into
mainfrom
llois-c/docs_adds_claude.md_files

Conversation

@lucasloisp

@lucasloisp lucasloisp commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds CLAUDE.md files to give AI assistants (Claude Code and similar) structured context about the repo.

  • CLAUDE.md (root) — repo map, commit conventions, release process, and a pointer to SDK/CLAUDE.md
  • SDK/CLAUDE.md — SDK-specific guide covering package structure, the interface/implementation architecture pattern, DI wiring, code formatting command, XML doc requirements, and pointers to existing docs for versioning, native plugins, and DI details
  • SampleApp/CLAUDE.md — one-liner describing what the sample app is
  • docs/dependency-injection.md — new doc covering constructor injection, wiring in PrivyImpl, and null-checking patterns

Kept intentionally lean: conventions enforced by the linter/editorconfig are not duplicated here, and sections that would restate existing docs/ content link out instead.

Testing

Reviewed content against agent_docs/pr_review_rules.md, docs/releasing.md, docs/native-code.md, and the existing SDK source to verify accuracy. Ran a Codex adversarial review — no findings.

Public package change?

  • Update the CHANGELOG.md file.

@lucasloisp lucasloisp requested a review from jagtejsodhi July 1, 2026 21:20

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Committed-By-Agent: claude
@lucasloisp lucasloisp marked this pull request as ready for review July 1, 2026 21:26
@lucasloisp lucasloisp merged commit 1b5e753 into main Jul 2, 2026
8 checks passed
@lucasloisp lucasloisp deleted the llois-c/docs_adds_claude.md_files branch July 2, 2026 14:04
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.

2 participants