Skip to content

update docs: verified hyperlinks, used doc writing skill#1088

Merged
wrenj merged 1 commit intogoogle:mainfrom
wrenj:dd33
Apr 7, 2026
Merged

update docs: verified hyperlinks, used doc writing skill#1088
wrenj merged 1 commit intogoogle:mainfrom
wrenj:dd33

Conversation

@wrenj
Copy link
Copy Markdown
Collaborator

@wrenj wrenj commented Apr 7, 2026

No description provided.

@github-project-automation github-project-automation bot moved this to Todo in A2UI Apr 7, 2026
@wrenj wrenj requested review from gspencergoog, jacobsimionato, nan-yu, polina-c and zeroasterisk and removed request for nan-yu and zeroasterisk April 7, 2026 18:25
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request standardizes the project's documentation by fixing relative paths, improving phrasing, and ensuring consistent punctuation in lists. It also converts MkDocs admonitions into a blockquote-based format to support the build pipeline. Review feedback identifies several instances where the required blockquote prefix was missing or where alert headers were redundant, such as 'NOTE: Note'.

4. Run the tool → the response contains A2UI JSON that renders a recipe card

> ⚠️ **Note**
> NOTE: Note
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The alert header is redundant. "NOTE: Note" repeats the label. It should be simplified to just "NOTE".

Suggested change
> NOTE: Note
> NOTE

```

> ⚠️ **Tip**
> TIP: Tip
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The alert header is redundant. "TIP: Tip" repeats the label. It should be simplified to just "TIP".

Suggested change
> TIP: Tip
> TIP

Comment on lines +22 to +24
WARNING: Security Notice

This demo runs an A2A agent that uses Gemini to generate A2UI responses. The agent has access to your API key and will make requests to Google's Gemini API. Always review agent code before running it in production environments.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This alert is missing the blockquote prefix (> ). According to docs/scripts/README.md, the documentation build pipeline expects a blockquote-based syntax to correctly transform alerts into MkDocs admonitions.

Suggested change
WARNING: Security Notice
This demo runs an A2A agent that uses Gemini to generate A2UI responses. The agent has access to your API key and will make requests to Google's Gemini API. Always review agent code before running it in production environments.
> WARNING: Security Notice
>
> This demo runs an A2A agent that uses Gemini to generate A2UI responses. The agent has access to your API key and will make requests to Google's Gemini API. Always review agent code before running it in production environments.

Comment on lines +64 to +66
NOTE: Demo Running

If everything worked, you should see the web app in your browser. The agent is now ready to generate UI!
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This alert is missing the blockquote prefix (> ). The build pipeline requires blockquotes for proper admonition rendering.

Suggested change
NOTE: Demo Running
If everything worked, you should see the web app in your browser. The agent is now ready to generate UI!
> NOTE: Demo Running
>
> If everything worked, you should see the web app in your browser. The agent is now ready to generate UI!

Comment on lines +171 to +173
TIP: It's Just JSON

Notice how readable and structured this is? LLMs can generate this easily, and it's safe to transmit and render—no code execution required.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This alert is missing the blockquote prefix (> ). The build pipeline requires blockquotes for proper admonition rendering.

Suggested change
TIP: It's Just JSON
Notice how readable and structured this is? LLMs can generate this easily, and it's safe to transmit and render—no code execution required.
> TIP: It's Just JSON
>
> Notice how readable and structured this is? LLMs can generate this easily, and it's safe to transmit and render—no code execution required.

Comment on lines +97 to +98
IMPORTANT: Attention
This area is under active design. The developer ergonomics are not yet finalized.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

This alert is missing the blockquote prefix (> ). The build pipeline requires blockquotes for proper admonition rendering.

Suggested change
IMPORTANT: Attention
This area is under active design. The developer ergonomics are not yet finalized.
> IMPORTANT: Attention
>
> This area is under active design. The developer ergonomics are not yet finalized.

@wrenj wrenj merged commit 059592e into google:main Apr 7, 2026
10 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in A2UI Apr 7, 2026
@polina-c polina-c mentioned this pull request Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants