Skip to content

Gotobeta - #262

Merged
aojea merged 11 commits into
google:mainfrom
aojea:gotobeta
Aug 11, 2026
Merged

Gotobeta#262
aojea merged 11 commits into
google:mainfrom
aojea:gotobeta

Conversation

@aojea

@aojea aojea commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

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

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.

Code Review

This pull request introduces auto-generation of admin tokens and database passwords in the Helm chart, adds a responsive mobile layout with a toggleable sidebar drawer and responsive tables to the console UI, and hides unauthorized services from MCP discovery by handling service-level authorization denials with a new ErrAuthRejected error. The review feedback suggests adding defensive checks in the Helm templates to handle empty or malformed existing secrets, refactoring the sidebar functions in JavaScript to avoid potential linter warnings, and providing a fallback value for the token TTL input to prevent sending NaN in the payload.

Comment thread charts/sam-mesh/templates/secrets.yaml
Comment thread cmd/sam-console/public/app.js
Comment thread cmd/sam-console/public/app.js Outdated
@aojea
aojea merged commit 5c85ff8 into google:main Aug 11, 2026
17 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.

1 participant