chore: add 'Google ADK' to valid frontmatter tags#98
Conversation
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.
There was a problem hiding this comment.
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'] |
There was a problem hiding this comment.
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'
]
Summary
Adds
'Google ADK'to the valid tags list intest/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.mduses theGoogle ADKtag, 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 thetagsarray on line 10.Test plan
frontmatter.mdin mcp-cookbook PR updated tutorial for iOS Sync to show how to use Docker Compose #3