Skip to content

refactor(client): extract constants and clean up patterns#10

Merged
lostmygithubaccount merged 1 commit intomainfrom
cody/code-quality
Apr 5, 2026
Merged

refactor(client): extract constants and clean up patterns#10
lostmygithubaccount merged 1 commit intomainfrom
cody/code-quality

Conversation

@lostmygithubaccount
Copy link
Copy Markdown
Member

Summary

  • Extract repeated (200..300) HTTP status range to a SUCCESS_STATUS constant in client.rs
  • Remove trailing comma in get_duckling_config format string
  • Extract repeated trim().to_string() pattern to a trimmed() helper in auth.rs

🤖 Generated with Claude Code

- Extract repeated (200..300) status range to SUCCESS_STATUS constant
- Remove trailing comma in format string (get_duckling_config)
- Extract trim().to_string() pattern to trimmed() helper in auth

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lostmygithubaccount lostmygithubaccount enabled auto-merge (squash) April 5, 2026 22:20
@lostmygithubaccount lostmygithubaccount merged commit 7e70863 into main Apr 5, 2026
1 check passed
@lostmygithubaccount lostmygithubaccount deleted the cody/code-quality branch April 5, 2026 22:21
lostmygithubaccount added a commit that referenced this pull request Apr 13, 2026
Patch bump from 0.2.1 -> 0.2.2 via `bin/bump-version --patch`.
Regenerates Cargo.lock, crates/dkdc-md-cli-py/Cargo.lock, and uv.lock.

## Highlights

Fifteen commits merged since v0.2.1 (2026-03-22). Biggest items are
auth hardening, Python binding test coverage, and CI/release pipeline
stabilization after the transfer to the `dkdc-io` org.

## Fixes / hardening

- `fix(auth): trim whitespace from env var tokens and add hardening tests` (#9)
- `fix(ci): add local PyPI publish step` (#6)
- `fix: add contents:read to release-rust publish job permissions` (#5)
- `fix: add checkout ref to release workflows + remove target cache` (#2)
- `fix: use dkdc.io email for dkdc-io org` (#4)

## Refactors / tests

- `refactor(client): extract API paths and use typed HTTP headers` (#11)
- `refactor(client): extract constants and clean up patterns` (#10)
- `test(py): add Python binding tests` (#13)

## Chores / infra

- `chore(ci): migrate to org reusable workflows` (#7)
- `chore: fix CLAUDE.md symlink to AGENTS.md` (#7)
- `chore: update repo references after transfer to dkdc-io` (#1)
- `chore: update dependencies` (#12)
- `docs: fix token resolution order and remove stale script reference` (#8)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lostmygithubaccount lostmygithubaccount mentioned this pull request Apr 13, 2026
2 tasks
lostmygithubaccount added a commit that referenced this pull request Apr 13, 2026
Patch bump from 0.2.1 -> 0.2.2 via `bin/bump-version --patch`.
Regenerates Cargo.lock, crates/dkdc-md-cli-py/Cargo.lock, and uv.lock.

## Highlights

Fifteen commits merged since v0.2.1 (2026-03-22). Biggest items are
auth hardening, Python binding test coverage, and CI/release pipeline
stabilization after the transfer to the `dkdc-io` org.

## Fixes / hardening

- `fix(auth): trim whitespace from env var tokens and add hardening tests` (#9)
- `fix(ci): add local PyPI publish step` (#6)
- `fix: add contents:read to release-rust publish job permissions` (#5)
- `fix: add checkout ref to release workflows + remove target cache` (#2)
- `fix: use dkdc.io email for dkdc-io org` (#4)

## Refactors / tests

- `refactor(client): extract API paths and use typed HTTP headers` (#11)
- `refactor(client): extract constants and clean up patterns` (#10)
- `test(py): add Python binding tests` (#13)

## Chores / infra

- `chore(ci): migrate to org reusable workflows` (#7)
- `chore: fix CLAUDE.md symlink to AGENTS.md` (#7)
- `chore: update repo references after transfer to dkdc-io` (#1)
- `chore: update dependencies` (#12)
- `docs: fix token resolution order and remove stale script reference` (#8)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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