Skip to content

Commit 4c7b07b

Browse files
committed
docs: update submission checklist with current verified state
- 10 PRs merged (was 7) - .well-known/oauth-protected-resource now 200 (was 'needs wrangler deploy') - 16/16 production checks pass (added) - 42/42 local MCP tests pass (added) - chatgpt-app-submission.json: 5 positive + 3 negative (was 6 positive + 12 negative) - Removed obsolete Cloudflare Worker deploy step - Fixed step numbering (Steps 1-5) Signed-off-by: Michael Kantor <6068672+kantorcodes@users.noreply.github.com>
1 parent 3540bc3 commit 4c7b07b

1 file changed

Lines changed: 18 additions & 26 deletions

File tree

SUBMISSION-CHECKLIST.md

Lines changed: 18 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,14 @@
33
## Prerequisites (verified)
44

55
- [x] Production MCP endpoint live: `POST https://hol.org/api/guard/mcp` → 401 without Bearer
6-
- [x] OAuth metadata: `GET https://hol.org/api/guard/oauth` → 200 with correct scopes
7-
- [x] `.well-known/oauth-protected-resource` fix committed to Cloudflare Worker (needs `wrangler deploy`)
6+
- [x] OAuth metadata: `GET https://hol.org/api/guard/oauth` → 200 with correct scopes, PKCE S256, `resource_parameter_supported: true`
7+
- [x] `.well-known/oauth-protected-resource` → 200 (FIXED via registry-broker PR #159 + k8s ingress patch, no Cloudflare Worker deploy needed)
88
- [x] hol-guard v2.0.1037 published to PyPI with MCP server and pagination fix
99
- [x] Plugin `.mcp.json` merged in hol-guard-plugin PR #13
10-
- [x] All 7 PRs merged, 0 unresolved review threads
11-
- [x] Guard-test: 19/19 steps pass
10+
- [x] All 10 PRs merged across 4 repos (hol-guard #1404, #1431, #1433; portal #3942, #3950, #3963; plugin #13, #14, #15; registry-broker #159), 0 unresolved review threads
11+
- [x] Guard-test: 19/19 steps pass with teardown
1212
- [x] Local MCP: 3 tools (search, fetch, get_guard_status), all read-only
13-
- [x] `chatgpt-app-submission.json` generated (3 tools, 6 test cases, 12 negative tests)
13+
- [x] `chatgpt-app-submission.json` schema-compliant (3 tools with annotations + justifications, 5 positive + 3 negative test cases)
1414
- [x] Privacy policy URL: `https://hol.org/points/legal/privacy` → 200
1515
- [x] Terms URL: `https://hol.org/points/legal/terms` → 200
1616
- [x] App icon: `assets/icon.png` exists (non-zero size)
@@ -21,21 +21,11 @@
2121
- [x] Tool annotations: all readOnlyHint=true, destructiveHint=false, openWorldHint=false
2222
- [x] Sanitization verified: no UUIDs, paths, or secrets in tool output
2323
- [x] Fetch output ≤ 32 KiB, search results ≤ 20
24-
24+
- [x] 16/16 production verification checks pass
25+
- [x] 42/42 local MCP contract + security tests pass
2526
## Human Actions Required
2627

27-
### Step 1: Deploy Cloudflare Worker
28-
29-
```bash
30-
cd ~/CascadeProjects/hashgraph-online/cloudflare-workers/hol-registry-proxy
31-
npx wrangler deploy
32-
```
33-
34-
Verify: `curl https://hol.org/.well-known/oauth-protected-resource` returns JSON with `resource`, `authorization_servers`, `scopes_supported`.
35-
36-
Note: MCP clients can use `https://hol.org/api/guard/oauth` directly in the meantime.
37-
38-
### Step 2: Create ChatGPT App
28+
### Step 1: Create ChatGPT App
3929

4030
1. Go to ChatGPT developer settings (https://chatgpt.com/developers)
4131
2. Create a new app
@@ -47,7 +37,9 @@ Note: MCP clients can use `https://hol.org/api/guard/oauth` directly in the mean
4737
- **Callback URI**: (set to the ChatGPT-provided callback URL)
4838
4. Record the real `plugin_asdk_app...` app ID
4939

50-
### Step 3: Add `.app.json`
40+
Note: `.well-known/oauth-protected-resource` is already live at `https://hol.org/.well-known/oauth-protected-resource` → 200. No Cloudflare Worker deploy needed.
41+
42+
### Step 2: Add `.app.json`
5143

5244
Create `hol-guard-plugin/.app.json` with the real app ID:
5345

@@ -62,17 +54,17 @@ Add to `.codex-plugin/plugin.json`:
6254
"apps": "./.app.json"
6355
```
6456

65-
### Step 4: Upload `chatgpt-app-submission.json`
57+
### Step 3: Upload `chatgpt-app-submission.json`
6658

6759
Upload `chatgpt-app-submission.json` to the ChatGPT Apps submission form. The file contains:
68-
- App info (name, developer, URLs, icon, descriptions)
60+
- App info (display_name, subtitle, category: DEVELOPER_TOOLS)
6961
- MCP server config (endpoint, OAuth, scopes, PKCE)
70-
- 3 tool definitions with annotation justifications
71-
- 6 positive test cases
72-
- 12 negative test cases (auth, isolation, sanitization)
62+
- 3 tool definitions with annotations + justifications
63+
- 5 positive test cases
64+
- 3 negative test cases (auth, isolation, sanitization)
7365
- Reviewer instructions (data access, security model, privacy)
7466

75-
### Step 5: Verify in ChatGPT Developer Mode
67+
### Step 4: Verify in ChatGPT Developer Mode
7668

7769
Test the following flows:
7870
- [ ] OAuth authorize flow succeeds
@@ -88,7 +80,7 @@ Test the following flows:
8880
- [ ] Stale data indicator when data is outdated
8981
- [ ] Result links open signed-in HOL Guard pages
9082

91-
### Step 6: Submit
83+
### Step 5: Submit
9284

9385
- [ ] Run platform pre-submission checks
9486
- [ ] Resolve every real issue

0 commit comments

Comments
 (0)