Skip to content

feat(handoff): enable reports by default - #1255

Merged
Zxf-xufeng merged 3 commits into
masterfrom
codex/handoff-report-default-on
Aug 17, 2026
Merged

feat(handoff): enable reports by default#1255
Zxf-xufeng merged 3 commits into
masterfrom
codex/handoff-report-default-on

Conversation

@Zxf-xufeng

Copy link
Copy Markdown
Member

Which issue or RFC does this PR close?

Closes #1254

Rationale for this change

Handoff Report is already implemented, but its default opt-in setting makes the Dashboard page difficult to discover. The Dashboard should expose the existing Handoff Report interface by default while keeping the statistics Dashboard at / and preserving an explicit opt-out.

What changes are included in this PR?

  • Change HandoffReportConfig.enabled default from false to true.
  • Keep handoff_report.enabled=false as the explicit opt-out path.
  • Update the HTTP test to verify that the opt-out still prevents Report-owned tables from being created.

Are there any user-facing changes?

With default settings, an authenticated Dashboard deployment exposes the Handoff Report page and navigation entry. The existing statistics Dashboard remains at /; this change does not redirect / to /handoff-reports or automatically select the Handoff Report tab. Core Handoff behavior is unchanged.

How was this change tested?

  • env -u POWERCONTEXT_SERVER_AUTH_ENABLED -u POWERCONTEXT_SERVER_AUTH_TOKEN -u POWERCONTEXT_SERVER_DASHBOARD_ENABLED -u POWERCONTEXT_SERVER_DASHBOARD_SCOPES uv run pytest -q tests/test_dashboard.py tests/test_handoff_report_http.py
  • Result: 6 passed.

AI usage statement

Implemented and validated with OpenAI Codex (GPT-5).

@CLAassistant

CLAassistant commented Aug 17, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Teingi Teingi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Zxf-xufeng
Zxf-xufeng force-pushed the codex/handoff-report-default-on branch from fb2a052 to abe7d3c Compare August 17, 2026 11:34
@Zxf-xufeng
Zxf-xufeng merged commit c12e6e3 into master Aug 17, 2026
10 checks passed
@Zxf-xufeng
Zxf-xufeng deleted the codex/handoff-report-default-on branch August 18, 2026 03:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: show Handoff Report in Dashboard by default

3 participants