Sync default verbs in config with connector hardcoded list#36
Conversation
The config package's defaultVerbs was missing "archive" and "update", which are present in the connector's filterArgs default list. Align the two so the schema default matches what the connector will actually accept and use. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Claude finished @btipling's task in 1m 15s —— View job PR Review: Sync default verbs in config with connector hardcoded list
Verdict: Looks good ✅ The change is correct and well-scoped. The config's Both additions are alphabetically ordered, consistent with the existing style. The regenerated One observation (not blocking)There's a similar discrepancy with
This might be intentional (only sync space-level nouns by default, requiring users to opt in for more granular types like Architectural noteHaving two separate |
Summary
pkg/config/config.godefaultVerbswas missingarchiveandupdate, which are both present in the connector'sdefaultVerbsinpkg/connector/connector.go(used byfilterArgsto validate/expand input). Adding them so the advertised config default matches the set the connector will actually accept.config_schema.jsonto reflect the new default.Test plan
go vet ./...go test ./...make lintgo build ./...🤖 Generated with Claude Code