Skip to content

docs(spec): add port information to /reflect endpoint sections#3561

Merged
lpcox merged 2 commits into
mainfrom
docs/reflect-endpoint-ports
May 21, 2026
Merged

docs(spec): add port information to /reflect endpoint sections#3561
lpcox merged 2 commits into
mainfrom
docs/reflect-endpoint-ports

Conversation

@lpcox
Copy link
Copy Markdown
Collaborator

@lpcox lpcox commented May 21, 2026

Clarifies that GET /reflect is 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 /health and /metrics.

Changes:

  • §10.6: Added introductory paragraph documenting port availability
  • §11.3: Cross-references §10.6 for port details

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>
Copilot AI review requested due to automatic review settings May 21, 2026 22:15
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

Documentation Preview

Documentation build failed for this PR. View logs.

Built from commit 02caa9b

@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 /reflect port availability and management-port behavior.
  • Updated §11.3 to reference §10.6 for /reflect port 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

Comment thread docs/awf-config-spec.md Outdated
Comment on lines +624 to +626
(10000–10004). Each port returns reflection metadata for its own provider
adapter. The management port (10000, OpenAI) additionally serves `/health`
and `/metrics`.
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@lpcox
Copy link
Copy Markdown
Collaborator Author

lpcox commented May 21, 2026

@copilot address review feedback

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor

Copilot AI commented May 21, 2026

@copilot address review feedback

Addressed in 562b712. The spec now says /reflect returns the same aggregate payload on every provider port, and clarifies that only port 10000 serves /metrics plus the aggregate /health while other ports keep provider-local /health.

Copilot finished work on behalf of lpcox May 21, 2026 22:22
@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

  • ✅ GitHub API: 2 PR entries confirmed
  • ✅ GitHub check: playwright_check PASS
  • ✅ File verify: smoke-test file exists

Overall: PASS

💥 [THE END] — Illustrated by Smoke Claude

@github-actions
Copy link
Copy Markdown
Contributor

🔥 Smoke Test: Copilot BYOK (Offline Mode)

Test Result
GitHub MCP (list PRs)
GitHub.com connectivity
File write/read (smoke-test-copilot-byok-26256536959.txt)
BYOK inference (agent → api-proxy → api.githubcopilot.com)

Running in BYOK offline mode (COPILOT_OFFLINE=true) via api-proxy → api.githubcopilot.com

Author: @lpcox | Overall: PASS

🔑 BYOK report filed by Smoke Copilot BYOK

@github-actions
Copy link
Copy Markdown
Contributor

🤖 Smoke Test Results

Test Result
GitHub MCP connectivity
GitHub.com HTTP connectivity ⚠️ (pre-step output not resolved)
File write/read ⚠️ (pre-step output not resolved)

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.

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Codex: FAIL
✅ Merged PRs: fix(dind-probe): address review feedback from #3554; feat: auto-detect DinD split filesystem via sentinel probe
❌ Safe Inputs GH CLI: safeinputs-gh missing
✅ Playwright: GitHub title verified
❌ Tavily: no callable search tool exposed
✅ File/Bash, discussion comment, and npm build passed

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • registry.npmjs.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "registry.npmjs.org"

See Network Configuration for more information.

🔮 The oracle has spoken through Smoke Codex

@github-actions
Copy link
Copy Markdown
Contributor

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia 1/1 passed ✅ PASS
Bun hono 1/1 passed ✅ PASS
C++ fmt N/A ✅ PASS
C++ json N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world N/A ✅ PASS
.NET json-parse N/A ✅ PASS
Go color ok ✅ PASS
Go env ok ✅ PASS
Go uuid ok ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx All passed ✅ PASS
Node.js execa All passed ✅ PASS
Node.js p-limit All passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Note (Java): The default ~/.m2 directory was owned by root (permission denied). Resolved by using -Dmaven.repo.local=/tmp/gh-aw/agent/m2repo for Maven invocations.

Generated by Build Test Suite for issue #3561 · ● 5.5M ·

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results

  • Redis PING: ❌ (timeout/no response)
  • PostgreSQL pg_isready: ❌ (no response)
  • PostgreSQL SELECT 1: ❌ (timeout/no response)

Overall: FAILhost.docker.internal is not reachable from this runner environment. Service containers appear unavailable.

🔌 Service connectivity validated by Smoke Services

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: Gemini Engine Validation

Overall status: FAIL

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • localhost

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "localhost"

See Network Configuration for more information.

💎 Faceted by Smoke Gemini

@lpcox lpcox merged commit 9d7457e into main May 21, 2026
55 of 57 checks passed
@lpcox lpcox deleted the docs/reflect-endpoint-ports branch May 21, 2026 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants