Skip to content

Polish report producer disconnected state#32

Merged
DeliciousBuding merged 1 commit into
mainfrom
platform/report-producer-print-disconnected-20260515
May 15, 2026
Merged

Polish report producer disconnected state#32
DeliciousBuding merged 1 commit into
mainfrom
platform/report-producer-print-disconnected-20260515

Conversation

@DeliciousBuding
Copy link
Copy Markdown
Owner

Summary

  • show a public-safe Runtime disconnected producer context when job detail hydration fails
  • make report audit producer context print/export friendly by avoiding page breaks and removing log-tail scroll clipping in print
  • sync the Platform roadmap current baseline and remove the completed P1 audit-loop item

Validation

  • python scripts/check_public_boundary.py
  • npm --prefix apps/web run lint (passes with existing 12 warnings)
  • npm --prefix apps/web run test
  • npm --prefix apps/web run build
  • go -C apps/api-go test ./...
  • go -C apps/api-go build ./cmd/platform-api
  • git diff --check
  • Playwright smoke: /workspace/reports/black-box?view=audit&job=job_demo_004 shows Runtime disconnected without raw connection text

Copilot AI review requested due to automatic review settings May 15, 2026 08:52
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request implements a disconnected state for the report producer context when job details are unavailable, ensuring the UI provides clear feedback through localized warning messages and badges. The changes include updates to the data fetching logic to return an unavailable status on failure, UI enhancements with print-specific CSS optimizations, and comprehensive test coverage for these new states. I have no feedback to provide.

Copy link
Copy Markdown

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@DeliciousBuding DeliciousBuding merged commit e71572d into main May 15, 2026
6 of 7 checks passed
@DeliciousBuding DeliciousBuding deleted the platform/report-producer-print-disconnected-20260515 branch May 15, 2026 08:54
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.

2 participants