Eviltester/issue23#24
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Restructures the project into an npm-workspaces monorepo and introduces reusable “core” generation functionality that is consumed by a new Node CLI package, a REST API app, and an MCP stdio server, with Docker support for the API/MCP apps.
Changes:
- Add
@anywaydata/corewithgenerateFromTextSpecplus initial Node test coverage. - Add
@anywaydata/cli,@anywaydata/api(Express + OpenAPI + options endpoints), and@anywaydata/mcp(JSON-RPC over stdio) that consume the core. - Convert repo to npm workspaces, add Dockerfiles/compose, and update CI/scripts to run workspace verification.
Reviewed changes
Copilot reviewed 27 out of 28 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/core/tests/generateFromTextSpec.test.js | Adds node:test coverage for new core generator API. |
| packages/core/src/index.js | Implements generateFromTextSpec + supported formats export. |
| packages/core/package.json | Defines publishable @anywaydata/core workspace package metadata/scripts. |
| packages/cli/src/index.js | New Node CLI wrapper around @anywaydata/core. |
| packages/cli/package.json | Defines publishable @anywaydata/cli workspace package metadata/scripts. |
| package.json | Enables npm workspaces; expands lint/format scope; adds verify/test workspace scripts. |
| package-lock.json | Updates lockfile for workspaces and new dependencies/apps. |
| docker-compose.yml | Adds compose setup for API + MCP builds. |
| apps/mcp/src/mcp.test.js | Adds node:test coverage for MCP JSON-RPC tool listing/calls. |
| apps/mcp/src/index.js | Implements MCP server over stdio calling generateFromTextSpec. |
| apps/mcp/package.json | Defines publishable @anywaydata/mcp workspace package metadata/scripts. |
| apps/mcp/Dockerfile | Docker build/runtime image for MCP server. |
| apps/api/src/startup.test.js | Adds tests for API port parsing and startup fallback behavior. |
| apps/api/src/options.route.test.js | Adds tests for options/tips endpoints and option-default behavior. |
| apps/api/src/openapi.js | Adds OpenAPI document for REST API endpoints. |
| apps/api/src/index.js | Implements Express REST API + OpenAPI/Swagger UI + option default storage. |
| apps/api/src/generate.route.test.js | Adds tests for /v1/generate response formats and error handling. |
| apps/api/src/fromschema.route.test.js | Adds tests for /v1/generate/fromschema text/plain behavior. |
| apps/api/src/docs.route.test.js | Adds tests for /openapi.json and /docs. |
| apps/api/package.json | Defines publishable @anywaydata/api workspace package metadata/scripts. |
| apps/api/Dockerfile | Docker build/runtime image for REST API server. |
| README.md | Documents workspaces, CLI/API/MCP usage, Docker, and publishing flow. |
| AGENTS.md | Adds local verification expectations (npm run verify:local). |
| .husky/pre-push | Adds pre-push hook to run verify:local. |
| .github/workflows/node.js.yml | Updates CI to Node 20 and runs verify:ci. |
| .dockerignore | Adds dockerignore for node_modules, coverage, build outputs, etc. |
| .changeset/config.json | Adds Changesets config for workspace versioning/publishing. |
| .changeset/README.md | Adds Changesets usage documentation. |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…ntain permissions' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…id-table-editor into eviltester/issue23
|
Important Review skippedToo many files! This PR contains 196 files, which is 46 over the limit of 150. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (196)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
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
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.
Closes #23