Skip to content

Replace SchemaType with Type in beginner agents documentation #74

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
yu-iskw merged 1 commit into from
Dec 21, 2025
Merged

Replace SchemaType with Type in beginner agents documentation #74

yu-iskw merged 1 commit into from
Dec 21, 2025

Conversation

@shiraji
Copy link
Contributor

@shiraji shiraji commented Dec 19, 2025 •
edited by qodo-code-review bot
Loading

User description

Update @google/genai import to use Type instead of SchemaType for consistency with the latest API.

🤖 Generated with Claude Code

PR Type

Documentation

Description

  • Replace SchemaType with Type in beginner agents documentation

  • Update import statements from @google/genai package

  • Maintain consistency with latest Google GenAI API

  • Update all schema type references throughout code examples

Diagram Walkthrough

flowchart LR
  A["SchemaType imports"] -- "replaced with" --> B["Type imports"]
  C["SchemaType.OBJECT/STRING/NUMBER"] -- "updated to" --> D["Type.OBJECT/STRING/NUMBER"]
  E["calculator.ts example"] -- "uses new" --> D
  F["defaultAgentNodes.ts example"] -- "uses new" --> D
Loading

File Walkthrough
Relevant files
Documentation
02_beginner_agents.md
Replace SchemaType with Type throughout documentation       

docs/projects/02_beginner_agents.md

  • Replace SchemaType import with Type from @google/genai package
  • Update all schema type references in calculator tool declaration
    (OBJECT, STRING, NUMBER)
  • Update schema type references in default agent configuration
  • Update import statement documentation comment from SchemaType to Type
 +11/-11 

@shiraji @claude
Replace SchemaType with Type in beginner agents documentation
3376d40 
Update @google/genai import to use Type instead of SchemaType
for consistency with the latest API.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@shiraji shiraji requested a review from yu-iskw as a code owner December 19, 2025 05:51
@qodo-code-review
Copy link

qodo-code-review bot commented Dec 19, 2025

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
⚪
🎫 No ticket provided
  • Create ticket/issue
Codebase Duplication Compliance
⚪
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
🟢
Generic: Comprehensive Audit Trails

Objective: To create a detailed and reliable record of critical system actions for security analysis
and compliance.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Meaningful Naming and Self-Documenting Code

Objective: Ensure all identifiers clearly express their purpose and intent, making code
self-documenting

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Robust Error Handling and Edge Case Management

Objective: Ensure comprehensive error handling that provides meaningful context and graceful
degradation

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Secure Error Handling

Objective: To prevent the leakage of sensitive system information through error messages while
providing sufficient detail for internal debugging.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Secure Logging Practices

Objective: To ensure logs are useful for debugging and auditing without exposing sensitive
information like PII, PHI, or cardholder data.

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules

Generic: Security-First Input Validation and Data Handling

Objective: Ensure all data inputs are validated, sanitized, and handled securely to prevent
vulnerabilities

Status: Passed

Learn more about managing compliance generic rules or creating your own custom rules
Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@qodo-code-review
Copy link

qodo-code-review bot commented Dec 19, 2025

PR Code Suggestions ✨

No code suggestions found for the PR.

@yu-iskw yu-iskw merged commit ecc7b6d into yu-iskw:main Dec 21, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yu-iskw yu-iskw yu-iskw approved these changes

Assignees

No one assigned

Labels

Review effort 1/5

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@shiraji @yu-iskw