Open
Conversation
- Remove expired docs (roadmap, kubernetes, user-management-todo, image-generation) - Consolidate search-related guides (exa, tavily, search-fallback) - Update docs layout navigation to match current structure - Sync web app docs to use markdown files with path mapping - Update API, architecture, guides, and configuration docs
- Remove duplicate Postman collections (response-aio, response-agent-slide) - Add comprehensive API test suite (api.spec.ts) for auth, models, chat, conversations, media, and health checks - Add tests/e2e/automation/README.md with structure documentation - Update Playwright config with API-only project Current structure: - 18 Postman collections (reduced from 20) - 2 Playwright spec files (increased from 1) - API coverage: auth, models, chat, conversations, media, health
…nto feat/code-improve
Contributor
There was a problem hiding this comment.
Pull request overview
This PR appears to decommission the apps/platform Next.js application (admin panel + docs) by removing the app’s code, documentation content, build tooling, and related CI workflows, and then updating top-level repository docs to reflect the new structure.
Changes:
- Removed the
apps/platformapplication (admin UI + Fumadocs-based docs) and its supporting scripts/config. - Deleted large portions of MDX documentation and generated API reference content that lived under
apps/platform/content. - Removed CI workflows and release jobs that built/deployed the platform app (and also removed some service image build jobs), and updated repo documentation (README/AGENTS/CLAUDE) to match.
Reviewed changes
Copilot reviewed 137 out of 353 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/platform/src/app/admin/mcp-tools/page.tsx | Removed MCP tools admin page as part of platform app decommission. |
| apps/platform/src/app/admin/mcp-tools/components/mcp-tool-modal.tsx | Removed MCP tool edit modal with platform app removal. |
| apps/platform/src/app/admin/layout.tsx | Removed admin layout and access checks (platform app removed). |
| apps/platform/src/app/(private)/layout.tsx | Removed private layout wrapper (platform app removed). |
| apps/platform/src/app/(home)/page.tsx | Removed platform home page (platform app removed). |
| apps/platform/src/app/(home)/layout.tsx | Removed platform home layout (platform app removed). |
| apps/platform/source.config.ts | Removed Fumadocs config as docs site is removed. |
| apps/platform/source.config.mjs | Removed Fumadocs re-export stub with platform decommission. |
| apps/platform/scripts/test-import.ts | Removed fumadocs import test script (no longer needed). |
| apps/platform/scripts/setup-env.sh | Removed platform env setup helper script. |
| apps/platform/scripts/setup-env.bat | Removed Windows platform env setup helper script. |
| apps/platform/scripts/generate-fumadocs-openapi.ts | Removed OpenAPI→Fumadocs generation script with docs removal. |
| apps/platform/scripts/README.md | Removed platform scripts documentation. |
| apps/platform/postcss.config.mjs | Removed platform PostCSS config. |
| apps/platform/package.json | Removed platform package definition/scripts/deps. |
| apps/platform/next.config.mjs | Removed platform Next.js config (MDX integration). |
| apps/platform/eslint.config.mjs | Removed platform ESLint config. |
| apps/platform/content/docs/runbooks/monitoring.mdx | Removed runbook doc content from platform docs tree. |
| apps/platform/content/docs/runbooks/meta.json | Removed runbooks nav metadata (platform docs removed). |
| apps/platform/content/docs/runbooks/index.mdx | Removed runbooks index page (platform docs removed). |
| apps/platform/content/docs/quickstart.mdx | Removed platform-hosted quickstart doc. |
| apps/platform/content/docs/meta.json | Removed platform docs root navigation metadata. |
| apps/platform/content/docs/guides/user-settings-personalization.mdx | Removed guide from platform docs tree. |
| apps/platform/content/docs/guides/troubleshooting.mdx | Removed troubleshooting guide from platform docs tree. |
| apps/platform/content/docs/guides/services-template.mdx | Removed services template guide from platform docs tree. |
| apps/platform/content/docs/guides/monitoring.mdx | Removed monitoring guide from platform docs tree. |
| apps/platform/content/docs/guides/meta.json | Removed guides nav metadata (platform docs removed). |
| apps/platform/content/docs/guides/mcp-testing.mdx | Removed MCP testing guide from platform docs tree. |
| apps/platform/content/docs/guides/mcp-admin-interface.mdx | Removed MCP admin interface guide from platform docs tree. |
| apps/platform/content/docs/guides/kong-plugins.mdx | Removed Kong plugins guide from platform docs tree. |
| apps/platform/content/docs/guides/index.mdx | Removed guides index page (platform docs removed). |
| apps/platform/content/docs/guides/background-mode.mdx | Removed background mode guide from platform docs tree. |
| apps/platform/content/docs/guides/authentication.mdx | Removed authentication guide from platform docs tree. |
| apps/platform/content/docs/conventions/workflow.mdx | Removed workflow conventions doc from platform docs tree. |
| apps/platform/content/docs/conventions/meta.json | Removed conventions nav metadata (platform docs removed). |
| apps/platform/content/docs/conventions/conventions.mdx | Removed conventions quick reference from platform docs tree. |
| apps/platform/content/docs/conventions/architecture-patterns.mdx | Removed architecture patterns doc from platform docs tree. |
| apps/platform/content/docs/configuration/service-migration.mdx | Removed configuration migration doc from platform docs tree. |
| apps/platform/content/docs/configuration/meta.json | Removed configuration nav metadata (platform docs removed). |
| apps/platform/content/docs/configuration/kubernetes.mdx | Removed k8s config doc from platform docs tree. |
| apps/platform/content/docs/configuration/index.mdx | Removed configuration system doc from platform docs tree. |
| apps/platform/content/docs/configuration/docker-compose.mdx | Removed docker-compose config doc from platform docs tree. |
| apps/platform/content/docs/architecture/system-design.mdx | Removed architecture system design doc from platform docs tree. |
| apps/platform/content/docs/architecture/services.mdx | Removed architecture services doc from platform docs tree. |
| apps/platform/content/docs/architecture/security.mdx | Removed security architecture doc from platform docs tree. |
| apps/platform/content/docs/architecture/security-advanced.mdx | Removed advanced security doc from platform docs tree. |
| apps/platform/content/docs/architecture/observability.mdx | Removed observability doc from platform docs tree. |
| apps/platform/content/docs/architecture/meta.json | Removed architecture nav metadata (platform docs removed). |
| apps/platform/content/docs/architecture/index.mdx | Removed architecture index doc from platform docs tree. |
| apps/platform/content/docs/architecture/data-flow.mdx | Removed data flow doc from platform docs tree. |
| apps/platform/content/docs/api-reference/server-api/version/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/server-api/readyz/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/server-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/content/docs/api-reference/server-api/healthz/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/projects-api/projects/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/projects-api/projects/patch.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/projects-api/projects/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/projects-api/projects/delete.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/projects-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/content/docs/api-reference/model-api/models/providers/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/model-api/models/catalogs/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/model-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/content/docs/api-reference/meta.json | Removed API reference root nav metadata (platform docs removed). |
| apps/platform/content/docs/api-reference/conversations-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/content/docs/api-reference/conversations-api/conversations/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/conversations-api/conversations/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/conversations-api/conversations/delete.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/conversations-api/conv-public-id/items/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/conversations-api/conv-public-id/items/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/conversations-api/conv-public-id/items/delete.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/chat-completions-api/models/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/chat-completions-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/content/docs/api-reference/chat-completions-api/completions/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/chat-completions-api.mdx | Removed API overview page (platform docs removed). |
| apps/platform/content/docs/api-reference/backward-compatibility.mdx | Removed API compatibility doc (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/validate/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/validate-api-key/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/upgrade/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/revoke/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/refresh-token/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/content/docs/api-reference/authentication-api/me/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/logout/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/login/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/guest-login/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/callback/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/api-keys/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/api-keys/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/authentication-api/api-keys/delete.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-provider-api/providers/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-provider-api/providers/patch.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-provider-api/providers/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-provider-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/content/docs/api-reference/admin-model-api/models/provider-models/patch.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-model-api/models/provider-models/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-model-api/models/provider-models/bulk-toggle/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-model-api/models/catalogs/patch.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-model-api/models/catalogs/get.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-model-api/models/catalogs/bulk-toggle/post.mdx | Removed generated API reference page (platform docs removed). |
| apps/platform/content/docs/api-reference/admin-model-api/meta.json | Removed generated API reference nav metadata. |
| apps/platform/components.json | Removed shadcn/ui components config for platform app. |
| apps/platform/cli.json | Removed platform-local CLI alias config. |
| apps/platform/Dockerfile | Removed platform Docker build definition. |
| apps/platform/.yarnrc.yml | Removed platform Yarn config. |
| apps/platform/.source/source.config.mjs | Removed generated Fumadocs source config output. |
| apps/platform/.prettierrc.json | Removed platform Prettier config. |
| apps/platform/.gitignore | Removed platform-specific gitignore. |
| apps/platform/.env.example | Removed sample env for platform app. |
| apps/platform/.dockerignore | Removed platform dockerignore. |
| README.md | Updated repo docs to remove platform app + realtime references and adjust commands. |
| CLAUDE.md | Updated contributor/agent guide to reflect removal of platform app and optional services list changes. |
| AGENTS.md | Updated agent overview to reflect removal of platform app and some service references. |
| .github/workflows/services-release-prod.yml | Removed release builds for memory-tools/vector-store/realtime images. |
| .github/workflows/deprecated-ci-app-platform-prod.yml | Removed deprecated platform CI workflow file. |
| .github/workflows/ci-production-release.yml | Removed production release builds for memory-tools/vector-store/realtime/platform. |
| .github/workflows/ci-app-platform-dev.yml | Removed platform CI workflow file. |
| .github/workflows/app-platform-release-prod.yml | Removed platform release workflow file. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | Framework | Gin (HTTP), zerolog (logging), Wire (DI) | | ||
| | ORM | GORM + goose migrations | | ||
| | Database | PostgreSQL (ankane/pgvector:latest for API, postgres:18 for Keycloak) | | ||
| | Database | PostgreSQL (postgres:18 for Keycloak) | |
There was a problem hiding this comment.
This row now reads like PostgreSQL is only used for Keycloak, but this document still describes the Go services using an ORM (GORM) and migrations, which implies a primary service database as well. Consider updating the Database entry to explicitly cover the main application Postgres (and any extensions like pgvector if still in use), and separately note the Keycloak DB image/version.
Suggested change
| | Database | PostgreSQL (postgres:18 for Keycloak) | | |
| | Database | PostgreSQL for core services (GORM + goose migrations, optional pgvector); PostgreSQL (postgres:18) for Keycloak | |
| ```bash | ||
| # Start services | ||
| make up-full # Full stack (all 4 APIs + infrastructure) | ||
| make up-full # Full stack (all APIs + infrastructure) |
There was a problem hiding this comment.
The phrase "all APIs" is a bit ambiguous in a repo that has optional components/profiles; it may help readers if this comment names the intended set (e.g., "LLM/Response/Media/MCP + infrastructure") or clarifies whether optional profiles are included/excluded.
Suggested change
| make up-full # Full stack (all APIs + infrastructure) | |
| make up-full # Full stack (LLM/Response/Media APIs + MCP tools + infrastructure) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Code Improvement