Repository navigation
docs(glean): note that search may be named enterprise_search - #33
Closed
steve-calvert-glean wants to merge 1 commit into
Closed
steve-calvert-glean wants to merge 1 commit into
steve-calvert-glean wants to merge 1 commit into
Conversation
Some deployments serve the document search tool as `enterprise_search` instead of `search`. Add a pluginpack partial that says both names are the same tool, and include it once in each skill and rule that names the tool, so the wording lives in one place.
david-hamilton-glean
approved these changes
Oct 12, 2026
Contributor
Author
|
Closing because this is now obsolete. Since 1024d08, the plugin lists Glean search as |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Some deployments serve Glean's document search tool as
enterprise_searchinstead ofsearch. The bundled skills and the Cursor rule only saysearch, so on those deployments they name a tool that isn't in the tool list.using-gleanalso says tool names are "always consistent across deployments", which is no longer true.Change
partials/search-tool-name.md, that says both names are the same tool with the same parameters, and to use whichever one the tool list shows.source.partialsinpluginpack.config.ts.{{> search-tool-name}}once in each skill and rule that names the tool:using-glean(and itsreference/search.md),plan-prep,project-awareness,project-handoff,using-glean-code,using-glean-productivity,search,skill-creation-guide(shared and the Codex override), and the Cursorglean-tool-selectionrule.using-glean.Inline mentions of
searchstay as they are. One note per file covers them, and the source stays readable. When every deployment serves one name, we update the partial and rebuild.This complements #32, which handles the tool name in the local MCP server.
Testing
npm test(build +pluginpack validatefor claude, cursor, and codex) passes.dist/against a build ofmain: 28 files changed (9 skill files × 3 targets, plus the Cursor rule). Every change adds the note, except the one corrected sentence inusing-glean. No unresolved{{>tags remain indist/.