Skip to content

chore: update tooling, CI, docs, and tests#3

Merged
min0625 merged 1 commit into
mainfrom
chore/tooling-docs-tests
Jun 23, 2026
Merged

chore: update tooling, CI, docs, and tests#3
min0625 merged 1 commit into
mainfrom
chore/tooling-docs-tests

Conversation

@min0625

@min0625 min0625 commented Jun 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Devcontainer: Replace docker-in-docker with GitHub CLI feature; add mise/TOML VS Code extensions; harden post_create.sh with set -euo pipefail and local hook support
  • CI: Upgrade CodeQL to Advanced (add actions language); pin mise-action to commit SHA; add Go/golangci-lint caches and job timeout to pr-check; add new coverage.yml workflow with Codecov upload
  • Linting: Bump golangci-lint to v2.12.2 with many new linters (modernize, copyloopvar, mirror, perfsprint, etc.); add golines max-len=120; remove swaggo formatter
  • Tooling: Bump Go to 1.26.4; add .PHONY targets and cover/cover-html Makefile targets; add coverage files and local config patterns to .gitignore
  • Docs: Improve README structure; add codecov badge; add Traditional Chinese README (README.zh-TW.md); replace AGENTS.md with CLAUDE.md
  • Tests: Add round-trip tests (JSON/Text/SQL); add edge-case scenarios for invalid zones, Local rejection, and non-string JSON input

Test plan

  • make test passes with race detector
  • make lint passes (golangci-lint config verify + run)
  • make check (check-tidy + lint + test) passes
  • Coverage workflow uploads to Codecov on push to main

🤖 Generated with Claude Code

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@min0625 min0625 force-pushed the chore/tooling-docs-tests branch from 869ab87 to 1b5aec2 Compare June 23, 2026 03:00
- Devcontainer: replace docker-in-docker with GitHub CLI feature; add mise/TOML extensions; harden post_create.sh with set -euo pipefail and local hook support
- CI: upgrade CodeQL to Advanced (add actions language); pin mise-action; add Go/golangci-lint caches and timeout to pr-check; add coverage workflow with Codecov upload
- Linting: bump golangci-lint to v2.12.2 with many new linters; add golines max-len=120; remove swaggo formatter
- Tooling: bump Go to 1.26.4; add .PHONY targets and cover/cover-html Makefile targets
- Docs: improve README structure; add codecov badge; add Traditional Chinese README; rename AGENTS.md to CLAUDE.md
- Tests: add round-trip tests (JSON/Text/SQL); add edge-case scenarios for invalid zones, Local rejection, and type errors

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@min0625 min0625 force-pushed the chore/tooling-docs-tests branch from 1b5aec2 to 9fdc0f2 Compare June 23, 2026 03:04
@min0625 min0625 merged commit 34b1ad4 into main Jun 23, 2026
4 checks passed
@min0625 min0625 deleted the chore/tooling-docs-tests branch June 23, 2026 03:11
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