Repository navigation
Docs re-read: correct skills where the AgentBase docs differ (inbound auth header stripping, rate-limit fallback, memory limit, MCP) - #11
Merged
Conversation
… auth header stripping, rate-limit fallback, memory limit, MCP, vCR) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… as documented (verified live 2026-11-09) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…rincipals (verified live 2026-11-09) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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.
What
Re-read all 41 English pages of https://docs.greennode.ai/ai-stack/agent-base and corrected the skills where the docs say something different or the skills were silent on something an agent builder needs. Docs-only: no
assets/template code changed, descriptions untouched.Most important fix: in Runtime Inbound Auth
IAM PermissionsandJWTmode the platform stripsAuthorizationbefore the container (only No authorization forwards it), and the Console now defaults a new Runtime to IAM. The skills told the agent to read the user JWT fromAuthorizationbehind a Runtime JWT, which would 401 every request.Other items: key-level Rate Limit is shared by all models of the key (a fallback model does not bypass it), Memory event pagination
from≤ 5000 is documented, Outbound IP of a Private Runtime, vCRbackendName/ robot username / expiry, runtime env vars "non-sensitive config only", stateless contract, ephemeral filesystem (vStorage S3), gateway JWT settings (Discovery URL/JWKS, Principal claim), connector Header key/prefix, default scopes, M365 VNG bundle, Slack as Custom provider, 22 policy operators + AND, a Built-in Tools row in the decision guide. Full list inCHANGELOG.md(Unreleased).Verified live (2026-11-09, temporary resources, all deleted): deleting a Policy Group does not detach it from a gateway as the docs say (the gateway keeps the dangling
policyGroupId; unbind first withPATCH{"policyGroupId": null}), and the documented policy-group limits were not enforced on the test account (100 policies per group accepted, 429errors.quota.policyat 101; 43 groups). The skills now say so. Gateway identitygw-<id>and runtime identityruntime-<id>noted.Second live lab (2026-11-09, one account, one gateway, IAM inbound): a gateway with no Policy Group bound let every valid IAM caller's
tools/callthrough (skills said 403; a 2026-10 run saw 403) — now written as a security rule (bind a group before callers can reach the gateway). Bound group with zero policies or a dangling id ⇒ deny; a deny was HTTP 200 + MCP tool errorRequest denied by policy.(no 403 seen); deny wins over allow in any order; bareiam/jwtprincipals rejected by the API (400) — useiam:*,jwt:*,*; policy changes apply in seconds,PATCHbind needs UPDATING → ACTIVE; targets at<gateway endpoint>/<target name>. Not tested: JWT-inbound gateways, non-owner callers, conditions, a real Runtime service account.Not changed (needs a decision or a live check)
jwtmode and the Expo app send the user token only inAuthorization, so they cannot work behind a Runtime JWT; the skills now say so, the code is untouched (small fix, needs web/frontend tests and a dev check).grn/ the API default to for inbound auth is not documented;agent.yaml.tpl, the deploy skill and theci.ymlsmoke test still assume "No authorization" forgrn deploy up.REPORT-docs-sync.md).Checks
uv run -q --python 3.12 --no-project python3 .github/scripts/check_skills.py— green (includes drift checks).github/scripts/check_drift.py— green🤖 Generated with Claude Code