Skip to content

docs: add CLAUDE.md with codebase overview and development guide#3

Merged
Wahib-L merged 4 commits into
mainfrom
claude/add-claude-documentation-izF6m
Mar 19, 2026
Merged

docs: add CLAUDE.md with codebase overview and development guide#3
Wahib-L merged 4 commits into
mainfrom
claude/add-claude-documentation-izF6m

Conversation

@Wahib-L
Copy link
Copy Markdown
Member

@Wahib-L Wahib-L commented Mar 15, 2026

Provides AI assistants with project structure, build commands,
architecture patterns, code conventions, and CI/CD details.

https://claude.ai/code/session_01KzkmihiXtnU6koPk8jkfWH

claude added 4 commits March 15, 2026 18:26
Provides AI assistants with project structure, build commands,
architecture patterns, code conventions, and CI/CD details.

https://claude.ai/code/session_01KzkmihiXtnU6koPk8jkfWH
Tests:
- Add unit tests for parse_env_file, construct_url, get_base_url,
  get_access_token, build_table in helpers.rs
- Add Serde deserialization tests for all models (StackList,
  EndpointList, TeamList, User, Stack) and enums
- Add tempfile dev-dependency for test file handling

Critical fixes:
- Fix Dockerfile ENTRYPOINT path (/bin/wrpt -> /usr/bin/wrpt)
- Propagate API errors in start/stop/remove handlers instead of
  silently ignoring them (let _ = ... -> ...?)

Other fixes:
- Fix parse_env_file: preserve # in values, add quoted value support
- Add pub(crate) to all list model fields for consistency
- Check Option before dereference in resource_control handler
- Fix misleading variable name in get_access_token()

https://claude.ai/code/session_01KzkmihiXtnU6koPk8jkfWH
Architecture improvements:
- Introduce CliError enum for structured error propagation,
  replacing Result<T, ()> and log_expect() panics throughout
- Introduce CliContext struct that holds a reusable HTTP client
  and resolved config (base_url), created once at startup
- Add 30s timeout on HTTP client
- Add resolve_stack() helper to deduplicate stack lookup logic
  across start/stop/remove/resource_control handlers
- Warn when --insecure flag is used

Cleanup:
- Remove log_err dependency (no longer needed)
- Remove all log_expect/unwrap calls in favor of ? operator
- Simplify handler signatures: accept &CliContext instead of
  individual base_url/access_token/insecure params
- Net reduction of ~190 lines of code

Dockerfile hardening:
- Pin Rust version to 1.83
- Pin docker-compose-bin to v2.32.4
- Add non-root user
- Clean apt cache in all stages
- Use --no-install-recommends
- Remove unnecessary clippy/rustfmt from build stage

https://claude.ai/code/session_01KzkmihiXtnU6koPk8jkfWH
… and CI/CD guides

Add Installation section (cargo, Docker, from source), Quick Start guide,
and CI/CD integration examples for GitHub Actions and GitLab CI.
Update roadmap to mark Documentation and Testing as completed.
Update CLAUDE.md to reflect CliContext/CliError architecture.

https://claude.ai/code/session_01KzkmihiXtnU6koPk8jkfWH
@Wahib-L Wahib-L merged commit 7f76808 into main Mar 19, 2026
2 checks passed
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