| Files | Last commit | Last update |
|---|---|---|
test: address review findings for browser.Open scheme validation - Replace misleading custom contains helper with strings.Contains (P1) - Add TestOpenRejectsBadScheme to verify Open() enforces validation (P1) - Add TestValidateURLSchemeParseError for url.Parse error branch (P1) - Add boundary cases: userinfo, localhost, IPv6, mailto, protocol-relative, leading space, control char, null byte, scheme with space - Add scheme name assertion to error message test Refs #358 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> | 4 months ago | |
docs(actions): clarify full commit SHA filter Refs #512 | 2 months ago | |
fix: sanitize SSH key terminal output | 2 months ago | |
feat(repo): add gc repo set-default — set/view/clear default repository Add a new repo set-default subcommand that configures the default repository for the current host. Pure client-side, no API calls. Features: - No args: set from git remote context - <owner>/<repo>: set explicit default - --view: display current default (text or --json) - --unset: clear default - Stored in config.json under default_repo key (per-host) - GC_DEFAULT_REPO env var overrides config Config change: add "default_repo" to allowedConfigKeys in pkg/config/config.go (1-line addition). 11 unit tests cover: flags validation, set explicit/git-context/ empty/invalid, view empty/set/json, unset, empty string. Real command verification: 6/6 scenarios pass (help, set, view, view --json, unset, view-after-unset). Refs gitcode-cli/cli#524 | 2 months ago | |
fix(update): make npm updates explicit and transactional | 2 months ago | |
merge: resolve conflicts between feature/issue-404 and main Resolve merge conflicts in 4 files by adopting main's cleaner implementation while preserving PR branch's more comprehensive test coverage (flag registration test, TTY-based CanPrompt tests, nil-safe CanPrompt test). | 3 months ago | |
test(lark): cross-platform stubs for CI (fix #499) lark 测试硬编码 /bin/true 作 FindLarkCLI 桩，该路径仅 Linux 存在，导致 macOS/Windows CI 报 lark-cli not installed（4 测试跨平台失败，main HEAD 既有）。改用 os.Executable() （运行中的测试二进制，全平台存在且 isExecutable 通过）。另 TestSaveDefaultChat 的 0600 权限断言 Windows-only 失败（Go 在 Windows 报 0666），改为 unix-only 断言。 - doctor_test.go/install_test.go: /bin/true → existingExecutable(t) (os.Executable) - config_test.go: 0600 断言 runtime.GOOS != windows 时才校验 | 2 months ago | |
fix(pr): normalize merged state in JSON output GitCode may return merged as null for merged pull requests. Decoding that value into a Go bool produced false even when state was merged and merged_at was populated, giving agents contradictory JSON. Normalize PullRequest.Merged from the explicit flag, merged state, or a non-empty merged_at value at API boundaries. Keep the public field a bool so the JSON schema remains stable, and use the shared IsMerged predicate for text rendering. Apply the same normalization to merge responses. Add API and command regression coverage for pr view, pr list, and pr merge, and document the normalized JSON contract. Refs #484 | 3 months ago | |
fix(precommit): enforce pre-push hooks | 3 months ago | |
refactor: extract roundTripFunc to pkg/testutil Replace 17 duplicate roundTripFunc type definitions across test files with a single shared RoundTripFunc in pkg/testutil. - Add pkg/testutil/roundtrip.go with RoundTripFunc type and NewRoundTripFunc() - Update 18 test files to use testutil.NewRoundTripFunc() - All 1214 tests pass, build succeeds Refs #347 Co-Authored-By: Claude <noreply@anthropic.com> | 4 months ago |