Skip to content

chore: add 'Google ADK' to valid frontmatter tags#98

Merged
teetangh merged 1 commit intomainfrom
chore/add-google-adk-tag
Apr 14, 2026
Merged

chore: add 'Google ADK' to valid frontmatter tags#98
teetangh merged 1 commit intomainfrom
chore/add-google-adk-tag

Conversation

@teetangh
Copy link
Copy Markdown
Contributor

Summary

Adds 'Google ADK' to the valid tags list in test/test-markdown-frontmatter.js.

This is needed for the Google ADK + Couchbase MCP Server tutorial being added in couchbase-examples/mcp-cookbook#3. The tutorial's frontmatter.md uses the Google ADK tag, and the mcp-cookbook CI runs the frontmatter validation from this repo — which currently rejects the tag as invalid.

Change

One line: append 'Google ADK' to the tags array on line 10.

Test plan

Needed for the Google ADK + Couchbase MCP Server tutorial in
couchbase-examples/mcp-cookbook (PR #3). The tutorial uses the
'Google ADK' tag in its frontmatter.md and the CI frontmatter
validation rejects it without this addition.
@teetangh teetangh requested a review from a team as a code owner April 13, 2026 09:57
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 adds the 'Google ADK' tag to the tags array in test/test-markdown-frontmatter.js. Feedback suggests reformatting the array into a multi-line, alphabetically sorted list to improve maintainability and identifies 'Google ADK' as a potential typo for 'Google SDK' or 'Google AI SDK'.

const sdk_languages = ['nodejs', 'scala', 'python', 'swift', 'csharp', 'objective-c', 'android-java', 'any', 'java', 'kotlin', 'dart', 'golang', 'ruby', 'php', 'c++']

const tags = ['Ottoman', 'Ktor', 'REST API', 'Express', 'Flask', 'TLS', 'Configuration', 'Next.js', 'iOS', 'Xcode', '.NET', 'Xamarin', 'Authentication', 'OpenID', 'Keycloak', 'Android', 'P2P', 'UIKit', 'Installation', 'Spring Boot', 'Spring Data', 'Transactions', 'SQL++ (N1QL)', 'Optimization', 'Community Edition', 'Docker', 'Data Modeling', 'Metadata', 'Best Practices', 'Data Ingestion', 'Kafka', 'Airbyte', 'Support', 'Customer', 'Prometheus', 'Monitoring', 'Observability', 'Metrics', 'Query Workbench', 'ASP.NET', 'linq', 'DBaaS', 'App Services', 'Flutter', 'Gin Gonic', 'FastAPI', 'Rails', 'Couchbase ORM', 'Laravel', 'LangChain', 'OpenAI', 'Streamlit', 'Google Gemini', 'Nvidia NIM', 'LLama3', 'AWS', 'Artificial Intelligence', 'Cohere', 'Jina AI', 'Mistral AI', 'Ragas', 'Haystack', 'LangGraph', 'Amazon Bedrock', 'CrewAI', 'PydanticAI', 'C++', 'C++ SDK', 'smolagents', 'Ag2', 'Autogen', 'Couchbase Edge Server', 'Deepseek', 'OpenRouter', 'mastra', 'Looker Studio', 'Google Data Studio', 'Connector', 'Couchbase Columnar', 'TAVs', 'Custom Queries', 'Data API', 'FTS', 'GSI', 'Hugging Face', 'LlamaIndex', 'GraphQL', 'Semantic Kernel', 'Hyperscale Vector Index', 'Composite Vector Index', 'Search Vector Index', 'Model Context Protocol (MCP)']
const tags = ['Ottoman', 'Ktor', 'REST API', 'Express', 'Flask', 'TLS', 'Configuration', 'Next.js', 'iOS', 'Xcode', '.NET', 'Xamarin', 'Authentication', 'OpenID', 'Keycloak', 'Android', 'P2P', 'UIKit', 'Installation', 'Spring Boot', 'Spring Data', 'Transactions', 'SQL++ (N1QL)', 'Optimization', 'Community Edition', 'Docker', 'Data Modeling', 'Metadata', 'Best Practices', 'Data Ingestion', 'Kafka', 'Airbyte', 'Support', 'Customer', 'Prometheus', 'Monitoring', 'Observability', 'Metrics', 'Query Workbench', 'ASP.NET', 'linq', 'DBaaS', 'App Services', 'Flutter', 'Gin Gonic', 'FastAPI', 'Rails', 'Couchbase ORM', 'Laravel', 'LangChain', 'OpenAI', 'Streamlit', 'Google Gemini', 'Nvidia NIM', 'LLama3', 'AWS', 'Artificial Intelligence', 'Cohere', 'Jina AI', 'Mistral AI', 'Ragas', 'Haystack', 'LangGraph', 'Amazon Bedrock', 'CrewAI', 'PydanticAI', 'C++', 'C++ SDK', 'smolagents', 'Ag2', 'Autogen', 'Couchbase Edge Server', 'Deepseek', 'OpenRouter', 'mastra', 'Looker Studio', 'Google Data Studio', 'Connector', 'Couchbase Columnar', 'TAVs', 'Custom Queries', 'Data API', 'FTS', 'GSI', 'Hugging Face', 'LlamaIndex', 'GraphQL', 'Semantic Kernel', 'Hyperscale Vector Index', 'Composite Vector Index', 'Search Vector Index', 'Model Context Protocol (MCP)', 'Google ADK']
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 tags array is extremely long (over 1,500 characters), which makes it difficult to maintain and results in unreadable git diffs. Formatting it as a multi-line, alphabetically sorted array significantly improves readability and ensures that future additions or removals are clearly visible in diffs. Additionally, the tag 'Google ADK' appears to be a typo for 'Google SDK' or 'Google AI SDK', as 'ADK' typically refers to the Android Accessory Development Kit, which is unrelated to the Model Context Protocol (MCP) mentioned in the PR description.

const tags = [
  '.NET',
  'Ag2',
  'Airbyte',
  'Amazon Bedrock',
  'Android',
  'App Services',
  'Artificial Intelligence',
  'ASP.NET',
  'Authentication',
  'Autogen',
  'AWS',
  'Best Practices',
  'C++',
  'C++ SDK',
  'Cohere',
  'Community Edition',
  'Composite Vector Index',
  'Configuration',
  'Connector',
  'Couchbase Columnar',
  'Couchbase Edge Server',
  'Couchbase ORM',
  'CrewAI',
  'Custom Queries',
  'Customer',
  'Data API',
  'Data Ingestion',
  'Data Modeling',
  'DBaaS',
  'Deepseek',
  'Docker',
  'Express',
  'FastAPI',
  'Flask',
  'Flutter',
  'FTS',
  'Gin Gonic',
  'Google ADK',
  'Google Data Studio',
  'Google Gemini',
  'GraphQL',
  'GSI',
  'Haystack',
  'Hugging Face',
  'Hyperscale Vector Index',
  'Installation',
  'iOS',
  'Jina AI',
  'Kafka',
  'Keycloak',
  'Ktor',
  'LangChain',
  'LangGraph',
  'Laravel',
  'linq',
  'LlamaIndex',
  'LLama3',
  'Looker Studio',
  'mastra',
  'Metadata',
  'Metrics',
  'Mistral AI',
  'Model Context Protocol (MCP)',
  'Monitoring',
  'Next.js',
  'Nvidia NIM',
  'Observability',
  'OpenAI',
  'OpenID',
  'OpenRouter',
  'Optimization',
  'Ottoman',
  'P2P',
  'Prometheus',
  'PydanticAI',
  'Query Workbench',
  'Ragas',
  'Rails',
  'REST API',
  'Search Vector Index',
  'Semantic Kernel',
  'smolagents',
  'Spring Boot',
  'Spring Data',
  'SQL++ (N1QL)',
  'Streamlit',
  'Support',
  'TAVs',
  'TLS',
  'Transactions',
  'UIKit',
  'Xamarin',
  'Xcode'
]

@teetangh teetangh merged commit a4096d8 into main Apr 14, 2026
4 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