Skip to content

Docs/fix: correct syntax/NameError in authentication.md and standardize internal imports to public API across docs - #1993

Open
Syed-Umer-Ali wants to merge 1 commit into
google:mainfrom
Syed-Umer-Ali:docs/fix-internal-imports-and-broken-code
Open

Docs/fix: correct syntax/NameError in authentication.md and standardize internal imports to public API across docs#1993
Syed-Umer-Ali wants to merge 1 commit into
google:mainfrom
Syed-Umer-Ali:docs/fix-internal-imports-and-broken-code

Conversation

@Syed-Umer-Ali

Copy link
Copy Markdown
Contributor

What

Fixes 2 categories of issues across 19 files:

  1. Broken Python code — SyntaxError (mismatched quotes) and NameError
    (undefined variables) in docs/tools-custom/authentication.md.
  2. Internal module imports — 19 files import from
    google.adk.agents.llm_agent or google.adk.tools.tool_context instead of
    the public API (google.adk.agents, google.adk.tools).

Fix

  • 'YOUR_OAUTH_SCOPES"'YOUR_OAUTH_SCOPES'
  • oauth_client_idclient_id, oauth_client_secretclient_secret
  • All from google.adk.agents.llm_agent import Xfrom google.adk.agents import X
  • All from google.adk.tools.tool_context import ToolContextfrom google.adk.tools import ToolContext

Verification

Grep confirms zero remaining instances of the internal import patterns in
docs/.

@netlify

netlify Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit bf9d04c
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/6a611f00941c700009b39cdb
😎 Deploy Preview https://deploy-preview-1993--adk-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant