| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
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> | 1 天前 | |
fix(auth): require interactive token disclosure | 11 小时前 | |
fix(auth): require interactive token disclosure | 11 小时前 | |
refactor: move config from internal/ to pkg/ to fix layering Move internal/config → pkg/config to resolve the layering violation where pkg/cmdutil (shared pkg/ layer) imported internal/config (private internal/ layer). The config package is used by 16 files across all command packages and is inherently a shared component. Change: update all import paths from internal/config to pkg/config. No logic changes — pure mechanical refactor. Fixes #351 Co-Authored-By: Claude <noreply@anthropic.com> | 6 天前 | |
fix(auth): require interactive token disclosure | 11 小时前 | |
fix: deliver milestone 374842 issue batch | 2 个月前 | |
feat(precommit): emit structured install-failure result in --json mode Close a gap from PR #221 review (observation 1): when auto-install was authorized and EnsureTool failed, Check returned a hard error and the command layer returned before writing any JSON. A --json consumer got only exit 1 + stderr prose, with no machine-readable reason or failure category. - Add Reason install_failed and a structured Result.InstallFailureCategories ([]string: permission|network|toolchain, first-seen order, deduped). - EnsureTool now returns a typed *InstallError carrying the categories; Check fills Reason/InstallFailureCategories via errors.As before returning the err. - The check command writes the JSON body before returning the CLIError in --json mode (mirroring the not_in_repo path); human-readable detail still goes to stderr. - Tests: install-failed reason + categories at pkg and command layers; multi-category ordering; no-installer => toolchain. - Docs: COMMANDS.md and design spec document install_failed and install_failure_categories. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> | 29 天前 | |
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> | 6 天前 |