Skip to content

chore(plugins): remove non-functional Zomato Omi app#8930

Merged
kodjima33 merged 1 commit into
mainfrom
issues-improver/8905-remove-zomato-app
Jul 9, 2026
Merged

chore(plugins): remove non-functional Zomato Omi app#8930
kodjima33 merged 1 commit into
mainfrom
issues-improver/8905-remove-zomato-app

Conversation

@kodjima33

@kodjima33 kodjima33 commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

What

Removes the plugins/omi-zomato-app/ plugin entirely (FastAPI service + Railway deployment wiring: main.py, mcp_proxy.py, db.py, models.py, railway.toml, Procfile, requirements.txt, .gitignore).

Why

Per issue #8905, the Zomato app is non-functional and has never had a successful production deployment:

Total deployments checked: 1,885
Successful deployments found: 0
Earliest recorded deployment (4076445966) failed at 2026-03-15T20:15:16Z
Latest deployment (5295112471) failed at 2026-07-03T05:22:12Z

The issue explicitly frames this as a cleanup/removal, not a repair: "If the integration is still desired, it should probably be rebuilt as a new app with a successful deployment path."

What I verified

  • All zomato references in the repo were self-contained within plugins/omi-zomato-app/. After removal, git grep -il zomato returns no matches (exit 1).
  • No plugin registry, app-store listing, or shared config outside the plugin directory referenced the app — the plugin's Railway/deployment wiring (railway.toml, Procfile) lived inside the removed directory.
  • Pure deletion of dead code; no runtime paths in other services touch it.

fixes #8905

Auto-generated from issue feedback by the mini issues-improver. Review before merge.

Review in cubic

The omi-zomato-app plugin has never had a successful production
deployment (0 of 1,885 recorded deployments succeeded, Mar–Jul 2026).
It is dead code with no working deployment path. Remove the plugin
directory and its Railway wiring entirely; all references were
self-contained within plugins/omi-zomato-app/.

fixes #8905

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 8 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@Git-on-my-level Git-on-my-level added dependency-review Touches dependencies or lockfiles; needs dependency review needs-maintainer-review Needs human maintainer review before merge labels Jul 3, 2026

@Git-on-my-level Git-on-my-level left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maintainer review (comment-only, not a formal approval):

Thanks for cleaning this up. I reviewed the PR statically against issue #8905. The diff is a pure deletion of plugins/omi-zomato-app/ including its FastAPI service, MCP/OAuth proxy, Redis/file token storage, Railway config, Procfile, requirements, and local gitignore. I also checked the PR head for remaining zomato / omi-zomato-app references and found none.

Positive signal: this looks like a coherent cleanup of a non-functional, self-contained app, and the current checks are green.

I’m leaving this as comment-only rather than approving because it removes an integration and dependency/deployment surface; a human maintainer should confirm that deleting the app is the intended product direction before merge. If that intent is confirmed, I do not see code-level blockers in this PR.

@kodjima33 kodjima33 merged commit b74e601 into main Jul 9, 2026
3 checks passed
@kodjima33 kodjima33 deleted the issues-improver/8905-remove-zomato-app branch July 9, 2026 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependency-review Touches dependencies or lockfiles; needs dependency review needs-maintainer-review Needs human maintainer review before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove non-functional Zomato Omi app

2 participants