fix(deps): update github.com/bytebase/omni digest to 726bb7f#20992
fix(deps): update github.com/bytebase/omni digest to 726bb7f#20992renovate[bot] wants to merge 1 commit intomainbytebase/bytebase:mainfrom renovate/github.com-bytebase-omni-digestbytebase/bytebase:renovate/github.com-bytebase-omni-digestCopy head branch name to clipboard
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ed462c90e4
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| github.com/bytebase/gomongo v0.0.0-20260510192859-8526a440b352 | ||
| github.com/bytebase/lsp-protocol v0.0.0-20250324071136-1586d0c10ff0 | ||
| github.com/bytebase/omni v0.0.0-20260716044319-2538875143b8 | ||
| github.com/bytebase/omni v0.0.0-20260720033410-726bb7ff40aa |
There was a problem hiding this comment.
Strip psql metacommands before accepting them
This omni bump pulls in the PG change that treats top-level psql backslash commands such as \echo/\g as trivia, so a script like SELECT \echo hi 1; can now pass parsing/review. The PostgreSQL execution path still sends command.Text/stmt.Text unchanged to the server in backend/plugin/db/pg/pg.go (transactional, non-transactional, and autocommit paths), and the server does not understand psql metacommands, so these reviewed rollouts fail at execution. Please either strip the new omnipg.StripTopLevelMetacommands output before executing or keep diagnosing these commands as unsupported in Bytebase-managed migrations.
Useful? React with 👍 / 👎.
This PR contains the following updates:
2538875→726bb7fWarning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Configuration
📅 Schedule: (UTC)
* * 31 2 *)🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.