Skip to content

enhance: add discovery revision to server config - #16

Merged
thedadams merged 1 commit into
obot-platform:mainfrom
thedadams:push-zrwnwqnoxzqk
Sep 16, 2026
Merged

thedadams merged 1 commit into
obot-platform:mainfrom
thedadams:push-zrwnwqnoxzqk

Conversation

@thedadams

Copy link
Copy Markdown
Member

The discovery revision is an optional parameter that's only use is as a "cache buster" to force re-discovery of tools, prompts, etc.

Issue: obot-platform/obot#7918

The discovery revision is an optional parameter that's only use is as a
"cache buster" to force re-discovery of tools, prompts, etc.

Signed-off-by: Donnie Adams <donnie@obot.ai>
Copilot AI balanced review requested due to automatic review settings September 16, 2026 03:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No unresolved issues were identified.

Pull request overview

Adds an optional per-server discoveryRevision cache buster to trigger rediscovery of tools, prompts, and related catalog state.

Changes:

  • Documents discoveryRevision.
  • Loads the field into server configuration.
  • Adds regression coverage for revision-triggered rediscovery.
File summaries
File Description
README.md Documents revision behavior.
config/load.go Loads the revision field.
config/config.go Defines YAML and runtime fields.
catalog/registry_test.go Tests revision-triggered rediscovery.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Lite (auto)

Note

Copilot is running an experiment and ran this review at Lite.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@thedadams
thedadams merged commit 8b65b04 into obot-platform:main Sep 16, 2026
4 checks passed
@thedadams
thedadams deleted the push-zrwnwqnoxzqk branch September 16, 2026 13:08
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.

3 participants