Skip to content

Commit 07bdfb1

Browse files
authored
Merge pull request #3 from MachineWisdomAI/codex/mcp-review-repairs
fix(mcp): address workspace, runtime and release review comments
2 parents 137c96a + fc40c24 commit 07bdfb1

94 files changed

Lines changed: 618 additions & 2863 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.dockerignore‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,16 @@
1414
**/.env.local
1515
**/.env.*.local
1616
apps/frontend/google-service-account
17-
# Cloudflare Worker local state and secrets (apps/ansari-faithful-answers)
17+
# Cloudflare Worker local state and secrets (apps/mcp)
1818
**/.dev.vars
1919
**/.dev.vars.*
2020
**/.wrangler
2121
**/dist-worker
22+
23+
# Credentials and private environment files; committed public examples remain available.
24+
**/.npmrc
25+
**/.env.*
26+
!**/.env.ci
27+
!**/.env.example
28+
**/test-results
29+
**/playwright-report

‎.github/workflows/faithful-answers.yml‎

Lines changed: 0 additions & 32 deletions
This file was deleted.

‎.github/workflows/mcp.yml‎

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
name: MCP application
2+
on:
3+
pull_request:
4+
paths: &mcp-paths
5+
- 'apps/mcp/**'
6+
- 'pnpm-lock.yaml'
7+
- 'pnpm-workspace.yaml'
8+
- 'package.json'
9+
- 'LICENSE'
10+
- '.dockerignore'
11+
- '.github/workflows/mcp.yml'
12+
- 'packages/eslint-config/**'
13+
push:
14+
branches: [main, develop]
15+
paths: *mcp-paths
16+
permissions:
17+
contents: read
18+
jobs:
19+
mcp:
20+
name: MCP application
21+
runs-on: ubuntu-latest
22+
timeout-minutes: 20
23+
steps:
24+
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
25+
with:
26+
persist-credentials: false
27+
- uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
28+
with:
29+
node-version: '22'
30+
- name: Install pnpm
31+
run: corepack enable && corepack prepare pnpm@10.33.0 --activate
32+
- name: Install Chromium system dependencies
33+
run: pnpm dlx playwright@1.64.0 install-deps chromium
34+
- name: Install from the workspace lockfile
35+
run: pnpm install --frozen-lockfile --strict-peer-dependencies --filter ansari-mcp...
36+
- name: Verify the MCP application
37+
run: |
38+
pnpm mcp exec playwright install chromium
39+
pnpm mcp verify
40+
- name: Verify the root-context container
41+
run: bash apps/mcp/scripts/verify-container.sh

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,4 +66,4 @@ issues for vulnerabilities.
6666

6767
## Faithful Answers MCP application
6868

69-
[`apps/ansari-faithful-answers`](apps/ansari-faithful-answers/README.md) exposes Ansari answers through MCP with a self-contained React card for assistant hosts. It uses public saga a2ui packages, embedded Kitab fonts and local artwork. Start it separately with `pnpm faithful-answers build` then `pnpm faithful-answers start`; see its README for environment variables and its standalone Docker build.
69+
[`apps/mcp`](apps/mcp/README.md) exposes Ansari answers through MCP with a self-contained React card for assistant hosts. It uses public saga a2ui packages, embedded Kitab fonts and local artwork. Start it separately with `pnpm mcp build` then `pnpm mcp start`; see its README for environment variables and its standalone Docker build.

‎apps/ansari-faithful-answers/.dockerignore‎

Lines changed: 0 additions & 11 deletions
This file was deleted.

‎apps/ansari-faithful-answers/Dockerfile‎

Lines changed: 0 additions & 14 deletions
This file was deleted.

‎apps/ansari-faithful-answers/assets/NOTICE.md‎

Lines changed: 0 additions & 16 deletions
This file was deleted.

0 commit comments

Comments
 (0)