Skip to content

final audit of ADK docs#464

Merged
jacksonyzj merged 14 commits intoaj/adk-docs-updatefrom
jackson-audit-adk-docs-v2
Apr 23, 2026
Merged

final audit of ADK docs#464
jacksonyzj merged 14 commits intoaj/adk-docs-updatefrom
jackson-audit-adk-docs-v2

Conversation

@jacksonyzj
Copy link
Copy Markdown
Contributor

Summary

Audited each ADK section against the latest source, fixed inaccuracies, and documented features that had shipped but weren't in the docs. Updated tone and flow to better match the docs style, and refreshed the dev console screenshots across pages.

adkah and others added 12 commits April 21, 2026 21:17
Audited each ADK section against the agent-lack source repo and the
skills repo. Fixed incorrect content, documented missing features,
tightened prose, removed trailing nav Cards, and refreshed dev console
screenshots.

Corrections:
- CLI reference: added --profile global flag, corrected --format
  availability, added subsections for adk search, adk mcp, adk
  telemetry, adk theme
- HITL: switched integration deps to string shorthand (object form is
  deprecated), reworded dev-mode warning to match the real constraint
- Skills: matched install command args to source (-a codex claude-code),
  added adk_init_project to the MCP tools table
- Evals: fixed Eval import path (@botpress/runtime -> @botpress/adk)
  and evals directory (src/evals -> evals), documented event/
  expectSilence turns and idleTimeout
- Debugging: removed bogus 'adk logs "webhook"' content-filter example,
  added since= and limit= tokens
- Scripts: updated flag pass-through example to use -- separator
- Actions: fixed callAction curl (chat.botpress.cloud ->
  api.botpress.cloud, added Authorization header)
- Triggers: removed incorrect client parameter from handler table

Documented missing features:
- Tables: computed columns, \$options regex modifier, keyColumn,
  waitComputed
- Actions: cached prop
- Knowledge: search() method, Directory source options, description role
- Zai: summarize format/intermediateFactor/sliding, answer() response
  types, rate() multi-criteria, group() initialGroups/maxGroups

Flow and structure:
- Rewrote evals with cleaner ordering
- Restructured knowledge page with dev console inlined after Create
- Moved action-vs-tool comparison to the asTool subsection
- Tightened intros on knowledge and actions pages
- Inlined dev console mentions into Create sections across pages
- Removed trailing next-page Cards from nearly every page

Screenshots updated: actions, agent-steps, evals, knowledge, logs,
tables, traces, triggers.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
…ctions

Continues the audit pass. Fixed inaccuracies, documented missing
features, tightened prose, inlined dev console mentions, and refreshed
screenshots.

Workflows:
- Added state prop, workflow.fail, workflow.execute
- Added step.progress, noted getOrCreate default statuses
- Rewrote ambiguous step-name uniqueness paragraph
- Updated request.workflow.provide examples to pass request.step
- Inlined workflow dev console mention into Create section

Conversations:
- Fixed tools title casing (Define Tools -> Define tools)
- Clarified customComponent vs custom message types
- Added execute() signal? prop (AbortSignal cancellation)
- Noted nudge.interval default and max-unlimited behavior
- Replaced Chat page screenshot with agent-steps screenshots
- Fixed webchat integration install note (no auto-prompt)
- Replaced duplicated Traces blurb with link to debugging page

Setup:
- Fixed swapped default model claim in configuration
- Fixed evals.idleTimeout unit (seconds -> milliseconds)
- Title casing: Agent Configuration / Environment Setup -> sentence case
- Removed evals row pointing at src/evals, updated to evals/
- Trimmed project structure table to core files only
- Reworked integrations install vs configure framing
- Enable/disable example now leads with string shorthand
- Inlined secrets and config dev console mentions into existing sections

Screenshots refreshed: workflows, environment selector, secrets,
config variables, integrations hub, integration config.

Removed trailing next-page Cards from every page in these sections.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@jacksonyzj jacksonyzj requested a review from adkah as a code owner April 23, 2026 18:58
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

vale

adk/testing/scripts.mdx|46 col 79| [Vale.Spelling] Did you really mean 'args'?
adk/testing/scripts.mdx|46 col 121| [Vale.Spelling] Did you really mean 'args'?
adk/zai/classify.mdx|20 col 26| [Vale.Spelling] Did you really mean 'boolean'?

Comment thread adk/advanced/hitl.mdx
</Info>

HITL (Human-in-the-Loop) lets your agent hand off a conversation to a live human agent. This is powered by two dependencies working together: the **HITL integration** (provides the transport to a helpdesk or agent platform) and the **HITL plugin** (provides the actions your code calls).
HITL (Human-in-the-Loop) lets your agent hand off a conversation to a live human agent. It's powered by two dependencies working together: the **HITL integration** (the transport to a helpdesk or agent platform) and the **HITL plugin** (the actions your code calls).
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'helpdesk'?

Comment thread adk/ai-native/skills.mdx
Comment thread adk/cli-reference.mdx
Comment thread adk/cli-reference.mdx
Comment thread adk/cli-reference.mdx
Comment thread adk/testing/evals.mdx
})
```

## Organize evals
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'evals'?

Comment thread adk/testing/evals.mdx

## Organize evals

Evals have two organizing fields that pair with CLI filters: `tags` and `type`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'Evals'?

Comment thread adk/testing/evals.mdx

Evals have two organizing fields that pair with CLI filters: `tags` and `type`.

**Tags** are free-form labels. Use them to group evals you want to run together (smoke tests, critical paths, slow suites):
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'evals'?

Comment thread adk/testing/evals.mdx
adk evals --tag smoke # Run only evals tagged "smoke"
```

**Type** marks what the eval is for:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'eval'?

Comment thread adk/testing/evals.mdx
adk evals --type regression # Run only regression evals
```

## Run evals
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'evals'?

Incorporates PR #461 (adrian-kahali):
- Get Started restructure: welcome -> introduction, new top-level
  quickstart, updated docs.json sidebar
- Configuration page: <Tip> blocks linking to related guides,
  expandable config snippet, cleaner prose
- Environment setup: consolidated secrets + config explanations,
  better wording
- Integrations: clearer intro, CLI section labelled
- Conversations: <CodeGroup> for channel matching, <Tip> blocks
  for cross-references
- AI execution: <Note> for LLMz reference, tighter phrasing
- Tools: highlight annotations on code blocks

Conflicts resolved in favor of keeping the audit fixes from this
branch (correct default model order, ms units for idleTimeout,
evals/ path, agent-steps screenshot, no trailing Cards, no
standalone Dev console sections, "Botpress Cloud" wording, real
CLI reference secrets table instead of placeholder link) while
taking Adrian's <Tip>/<Note> block styling and structural
improvements.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
@jacksonyzj jacksonyzj changed the title docs(adk): final audit of ADK docs final audit of ADK docs Apr 23, 2026
Comment thread adk/quickstart.mdx
- Chat with your agent
- Browse its components (actions, workflows, triggers)
- Manage data (tables, knowledge bases, files)
- Run evals
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'evals'?

Comment thread adk/testing/scripts.mdx
## Arguments

Extra arguments after the script path are passed to your function:
Extra arguments after the script path are passed to your function. Positional args pass through directly; put flag-like args (anything starting with `-`) after a `--` separator so Commander doesn't try to parse them:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'args'?

Comment thread adk/testing/scripts.mdx
## Arguments

Extra arguments after the script path are passed to your function:
Extra arguments after the script path are passed to your function. Positional args pass through directly; put flag-like args (anything starting with `-`) after a `--` separator so Commander doesn't try to parse them:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'args'?

Comment thread adk/zai/classify.mdx
// true
```

Call `.result()` for the boolean plus the LLM's reasoning:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [vale] reported by reviewdog 🐶
[Vale.Spelling] Did you really mean 'boolean'?

Introduction:
- Tightened intro paragraph
- Added Custom components to primitives table
- Dropped src/ qualifier since evals live at the project root

Quickstart:
- Swapped em-dash for colon in the requirements list
- Dropped filler sentence at the top of Test your agent
- Removed the 'Chat in the terminal' section so readers land in the
  dev console
- Normalized smart apostrophe

Refreshed welcome and quickstart screenshots.
@jacksonyzj jacksonyzj merged commit 6f35f1f into aj/adk-docs-update Apr 23, 2026
7 checks passed
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.

2 participants