fix(i18n): 对齐 locale key schema 并补充 gosec 注释 #104

Merged
wbtiger merged 1 commits from muel/gitlink-cli:fix/i18n-schema-gosec into master 2026-06-08 02:19:20 +08:00
Contributor

本 PR 是已合入 i18n 基础设施 PR 的 follow-up 小修复,用于处理 review 中提到的两个 minor 问题。

变更内容

1. 对齐 locale key schema 与 validator

validate.go 已经允许 confirm.* key 前缀,但 messages.schema.jsonpropertyNames.pattern 中未包含 confirm

本 PR 将 confirm 补充到 schema pattern 中,使 schema 与 validator 对 locale key 前缀的约束保持一致。

2. 为 repo-local glob 用法补充 gosec 注释

i18n check 工具使用固定的仓库内 glob pattern 查找 locale JSON 文件。

本 PR 为 filepath.Glob 用法补充了针对性的 #nosec G703 注释,并说明该 pattern 来自仓库内固定路径,不受用户输入控制。

修改范围

本 PR 仅修改:

  • internal/i18n/cmd/check/main.go
  • internal/i18n/schema/messages.schema.json

不改变运行时行为,不修改 locale 内容���不改变命令行为、shortcut 行为或 API 输出。

验证结果

本地已执行并通过:

go run ./internal/i18n/cmd/check
go run ./internal/i18n/cmd/check --scan-code
go build .
go vet ./...
本 PR 是已合入 i18n 基础设施 PR 的 follow-up 小修复,用于处理 review 中提到的两个 minor 问题。 ## 变更内容 ### 1. 对齐 locale key schema 与 validator `validate.go` 已经允许 `confirm.*` key 前缀,但 `messages.schema.json` 的 `propertyNames.pattern` 中未包含 `confirm`。 本 PR 将 `confirm` 补充到 schema pattern 中,使 schema 与 validator 对 locale key 前缀的约束保持一致。 ### 2. 为 repo-local glob 用法补充 gosec 注释 i18n check 工具使用固定的仓库内 glob pattern 查找 locale JSON 文件。 本 PR 为 `filepath.Glob` 用法补充了针对性的 `#nosec G703` 注释,并说明该 pattern 来自仓库内固定路径,不受用户输入控制。 ## 修改范围 本 PR 仅修改: - `internal/i18n/cmd/check/main.go` - `internal/i18n/schema/messages.schema.json` 不改变运行时行为,不修改 locale 内容���不改变命令行为、shortcut 行为或 API 输出。 ## 验证结果 本地已执行并通过: ```bash go run ./internal/i18n/cmd/check go run ./internal/i18n/cmd/check --scan-code go build . go vet ./...
muel added 1 commit 2026-06-03 13:57:51 +08:00
wbtiger merged commit 9c085fb246 into master 2026-06-08 02:19:20 +08:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Gitlink/gitlink-cli#104
No description provided.