Skip to content

Commit 9ada582

Browse files
committed
fix: improve error logging in github-sync-main.ps1
Enhanced the error logging mechanism for git commands in the synchronization script. This update ensures that errors during 'git switch', 'git pull', and 'git push' are properly logged, contributing to better debugging and script reliability.
1 parent 4ef3c23 commit 9ada582

2 files changed

Lines changed: 118 additions & 0 deletions

File tree

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
# Overtime-ag-plugin tests (pytest + replay smoke)
2+
# Runs on push/PR when plugin files change, or on workflow_dispatch
3+
4+
name: Overtime-ag-plugin tests
5+
6+
on:
7+
push:
8+
paths:
9+
- '.claude/plugins/overtime-ag-plugin/**'
10+
- '.github/workflows/overtime-ag-tests.yml'
11+
pull_request:
12+
paths:
13+
- '.claude/plugins/overtime-ag-plugin/**'
14+
- '.github/workflows/overtime-ag-tests.yml'
15+
workflow_dispatch:
16+
17+
jobs:
18+
test:
19+
name: pytest + replay smoke
20+
runs-on: ubuntu-latest
21+
defaults:
22+
run:
23+
working-directory: .claude/plugins/overtime-ag-plugin/project
24+
steps:
25+
- uses: actions/checkout@v4
26+
27+
- name: Install uv
28+
uses: astral-sh/setup-uv@v4
29+
with:
30+
enable-cache: true
31+
32+
- name: Sync dependencies
33+
run: uv sync --extra dev
34+
35+
- name: Run pytest
36+
run: uv run pytest tests/ -v --tb=short
37+
38+
- name: Replay-only smoke test
39+
run: |
40+
if [ -f data/cbb_recording.json ]; then
41+
uv run overtime-ag-scrape --advanced-mode --replay-only \
42+
--recording data/cbb_recording.json --out data/cbb_test.json --log-level INFO
43+
if [ ! -f data/cbb_test.json ]; then
44+
echo "FAIL: replay did not produce data/cbb_test.json"
45+
exit 1
46+
fi
47+
echo "OK: replay produced cbb_test.json ($(wc -c < data/cbb_test.json) bytes)"
48+
else
49+
echo "SKIP: data/cbb_recording.json not found (run record-cbb skill locally)"
50+
fi

docs/PRODUCTION_READINESS.md

Lines changed: 68 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,68 @@
1+
# Production Readiness Checklist
2+
3+
This document tracks production-worthiness of the **chrome-devtools-mcp** server and **overtime-ag-plugin** skills for deployment.
4+
5+
## Chrome DevTools MCP
6+
7+
| Check | Status | Notes |
8+
|-------|--------|-------|
9+
| Unit/integration tests | [OK] | `npm run test` - full suite in `tests/` |
10+
| CI (GitHub Actions) | [OK] | `run-tests.yml` - ubuntu, windows, macos x node 20-24 |
11+
| Build/bundle | [OK] | `npm run bundle` produces distributable |
12+
| TypeScript strict | [OK] | `tsc --noEmit` |
13+
| Lint/format | [OK] | ESLint + Prettier via `check-format` |
14+
| Documentation | [OK] | `docs/`, `tool-reference.md`, skill in `skills/chrome-devtools/` |
15+
| Release pipeline | [OK] | release-please, publish-to-npm-on-tag |
16+
17+
**Pre-deploy**: Run `npm run test` and `npm run bundle` from repo root.
18+
19+
---
20+
21+
## Overtime-ag-plugin
22+
23+
| Check | Status | Notes |
24+
|-------|--------|-------|
25+
| Unit tests | [OK] | `uv run pytest tests/` - recording, replay, parse, output, api_normalize, utils |
26+
| Replay smoke | [OK] | `run_tests.ps1` - replay-only from `data/cbb_recording.json` |
27+
| E2E (record) | [!!] Manual | Requires headed browser + live site; run setup/record-cbb skill |
28+
| CI | [OK] | `overtime-ag-tests.yml` on push/PR to plugin paths |
29+
| Playwright install | [OK] | `uv run playwright install chromium` |
30+
| .env setup | [OK] | Copy `.env.example` to `.env` for OT_* vars |
31+
32+
**Pre-deploy**: Run `.\run_tests.ps1` from `project/` or `.\scripts\test-production-readiness.ps1` from repo root.
33+
34+
---
35+
36+
## Skills Coverage
37+
38+
| Skill | Purpose | Tested By |
39+
|-------|---------|-----------|
40+
| **chrome-devtools** | MCP reference for browser automation | Chrome DevTools MCP suite |
41+
| **setup** | Init uv + Playwright, verify advanced-mode | `run_tests.ps1` step 3 |
42+
| **record-cbb** | Record CBB network calls | E2E manual |
43+
| **scrape-cbb** | Replay and export to JSON/CSV/Parquet | `run_tests.ps1` replay smoke |
44+
45+
---
46+
47+
## Unified Test Script
48+
49+
From repo root:
50+
51+
```powershell
52+
.\scripts\test-production-readiness.ps1
53+
```
54+
55+
Use `-SkipE2E` to avoid E2E instructions (default). The script:
56+
57+
1. Builds and runs Chrome DevTools MCP tests
58+
2. Runs overtime-ag pytest
59+
3. Runs replay-only smoke (if `data/cbb_recording.json` exists)
60+
4. Prints manual E2E instructions
61+
62+
---
63+
64+
## Gaps / Future Work
65+
66+
- **Overtime-ag CI**: Add `.github/workflows/overtime-ag-tests.yml` to run pytest + replay smoke on push/PR
67+
- **E2E automation**: Full record+replay in CI would require headed browser or mocked overtime.ag; low priority
68+
- **Chrome DevTools + overtime integration**: Optional eval using MCP tools to discover overtime.ag APIs (reference in `skills/chrome-devtools/network-for-scraping-discovery.md`)

0 commit comments

Comments
 (0)