🟠 [claude] /init again#38
Conversation
There was a problem hiding this comment.
Pull request overview
Updates CLAUDE.md to reflect the repository’s current Rust/sea-orm implementation status and architecture (modules, CLI, sync flow), along with updated developer workflow notes.
Changes:
- Updates “Project Status” items (what’s complete vs remaining).
- Expands Rust architecture/module structure and OpenSRS client capabilities described in the doc.
- Updates technology stack + sea-orm entity generation notes and adds CLI/domains.toml guidance.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - Database caching layer (sea-orm integration) | ||
| - Multi-customer support (via `domains.toml` and `sync` command) | ||
|
|
||
| Remaining: | ||
|
|
There was a problem hiding this comment.
The status list marks the database caching layer as complete, but later in this document there is still a section titled “Future Database Caching” describing caching as not yet implemented. Please update one of these so the document is internally consistent (either rename/update that later section or adjust the status here).
| - Database caching layer (sea-orm integration) | |
| - Multi-customer support (via `domains.toml` and `sync` command) | |
| Remaining: | |
| - Multi-customer support (via `domains.toml` and `sync` command) | |
| Remaining: | |
| - Database caching layer implementation (sea-orm integration) |
| ### domains.toml | ||
|
|
||
| Maps customer slugs to their domain lists. This file is gitignored (contains real customer data). A template with example entries is committed. Format: | ||
|
|
||
| ```toml | ||
| [domain_groups] | ||
| customer_slug = ["domain1.com", "domain2.org"] | ||
| ``` |
There was a problem hiding this comment.
This says “A template with example entries is committed”, but there doesn’t appear to be any committed domains.toml template/example file in the repository (only domains.toml is gitignored). Either add a committed example file (e.g., domains.toml.example) or adjust this sentence to reflect the current repo state.
| ```bash | ||
| just try # Run with credentials (default) | ||
| just try # Run sync with credentials (default) | ||
| just run_with_creds # Run with OpenSRS credentials from 1Password | ||
| just backtrace # Run with RUST_BACKTRACE=1 for detailed errors | ||
| ``` | ||
|
|
||
| The default `just try` runs `macaw sync`. To run other subcommands with credentials: | ||
|
|
||
| ```bash | ||
| fnox run -- cargo run -- list 2026 | ||
| fnox run -- cargo run -- contacts example.com --contact-type admin --first-name Jane ... | ||
| ``` |
There was a problem hiding this comment.
The docs say just try runs macaw sync, but just try currently executes cargo run with no subcommand (see justfile:19-21), and clap requires a subcommand. Either update the try recipe to run the sync subcommand by default, or change this documentation to match the actual behavior.
Done
Meta
(Automated in
.just/gh-process.just.)