docs(spec): add port information to /reflect endpoint sections#3561
Conversation
Clarify that GET /reflect is available on all provider ports (10000–10004), each returning reflection metadata for its own adapter. Note that port 10000 (OpenAI) is also the management port serving /health and /metrics. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Pull request overview
Updates the AWF config specification to clarify how to access the api-proxy /reflect introspection endpoint across the sidecar’s provider ports, and to cross-reference that detail from the max-runs section.
Changes:
- Added an introductory paragraph in §10.6 describing
/reflectport availability and management-port behavior. - Updated §11.3 to reference §10.6 for
/reflectport details.
Show a summary per file
| File | Description |
|---|---|
| docs/awf-config-spec.md | Documents /reflect port availability and adds a cross-reference from max-runs introspection. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
| (10000–10004). Each port returns reflection metadata for its own provider | ||
| adapter. The management port (10000, OpenAI) additionally serves `/health` | ||
| and `/metrics`. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@copilot address review feedback |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
✅ Smoke Test Results
Overall: PASS
|
🔥 Smoke Test: Copilot BYOK (Offline Mode)
Running in BYOK offline mode ( Author: @lpcox | Overall: PASS
|
🤖 Smoke Test Results
PR: docs(spec): add port information to /reflect endpoint sections — @lpcox (no assignees) Overall: INCONCLUSIVE — MCP connectivity confirmed; pre-step template variables were not substituted, so tests 2 & 3 could not be verified.
|
|
Smoke Codex: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "registry.npmjs.org"See Network Configuration for more information.
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Smoke Test Results
Overall: FAIL —
|
Smoke Test: Gemini Engine Validation
Overall status: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
Clarifies that
GET /reflectis available on all provider ports (10000–10004), with each port returning reflection metadata for its own adapter. Notes that port 10000 (OpenAI) is the management port also serving/healthand/metrics.Changes: