docs: consolidate pre-publish gate management via Inventor and MCP (replaces #750, #752) - #756
docs: consolidate pre-publish gate management via Inventor and MCP (replaces #750, #752)#756jordanc-relevanceai wants to merge 2 commits into
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
🎯 Vibe checkReviewed: 4 files (2 with issues, 2 clean) Scores
Score key: 🟢 9–10, 🟡 6–8, 🔴 1–5. Scores are a single overall judgment about the whole PR — not per file. ✨ Overall vibe: 🔧 Issues (16)
🧩 Component suggestions (2)
🏗️ Page structure (1)
✅ Clean files (2)
🔋 Credit usage
Files read: |
…ames - evals-custom-tool-check is at 0% rollout and has never been called, so the Custom Tool accordion, FAQ, billing note and cross-links are removed until it ships (same call as #742 made on the cost columns) - MCP tools are registered as relevance_get_pre_publish_status et al; add the missing relevance_ prefix to all four - eval-pre-publish-per-test-set-be is at full rollout; drop the rollout caveat Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Going to be included in and superseded by a Evals documentation update shortly |
Summary
Documents managing pre-publish quality gates via Inventor or MCP, consolidated from two open PRs that both edited
build/agents/build-your-agent/evals.mdx. Branched frommain.skip_checksbypass, and the fail-closed behaviorinvent.mdx— capability-card update noting Inventor can manage gate configurationReplaces these PRs (now closed)
Omitted: everything from #750
evals-custom-tool-check(flag 785246) is at 0% rollout and has never been called. No customer can create a Custom Tool check, so the accordion, the FAQ entry, the billing note, the check-type list change, and thetools.mdx/create-a-tool.mdxcross-links are all left out rather than caveated. Same call #742 made on the eval cost-breakdown columns.Worth keeping #750's text for when it ships — the contract it describes is accurate. Verified against
CustomToolEvalRuleForm.vue:236-243: aconversationinput, a booleanpassedoutput, an optionalreasonstring, and a Create evaluator tool scaffolding button. One correction to carry over: the code blocks saving the check when the tool drifts out of contract, it does not "fail until resolved" as #750 claimed.Corrections to #752
All four MCP tool names were missing their prefix. #752 documented
get_pre_publish_status,upsert_pre_publish_config,delete_pre_publish_configandpoll_pre_publish_check. They are registered asrelevance_get_pre_publish_status,relevance_upsert_pre_publish_config,relevance_delete_pre_publish_configandrelevance_poll_pre_publish_check(apps/nodeapi/src/mcp/tools/tool-names.ts). Fixed — as written, none of them would have worked.Dropped the flag key, and the rollout caveat with it. #752 named
eval-pre-publish-per-publish-per-test-set-be. No such flag exists — the real key iseval-pre-publish-per-test-set-be(flag 748191), and it is at 100% rollout. I initially replaced the key with a "rolling out progressively, contact your account manager" note; that was also wrong, so it's now removed entirely. The feature is fully available.Verified as accurate and kept
upsert_pre_publish_config—MAX_CONFIG_TEST_SETS = 10with.min(1)(apps/nodeapi/src/mcp/tools/eval-pre-publish.ts:26,48-50).skip_checksexists as a bypass parameter (apps/nodeapi/src/mcp/tools/agent.ts,workforce.ts).threshold_score(0–100) andblock_on_failure.Test plan
maininvent.mdxcapability card rendersNote on
evals.mdxcontentionThis page also has #742 and #747 open against it. This PR only touches the Publish section, so it should not conflict with either. #747 is separately blocked on a 0% flag.