Skip to content

Fix docs dev server network isolation and MonaSans preload warnings - #43725

Closed
pelikhan with Copilot wants to merge 5 commits into
mainfrom
copilot/multi-device-docs-testing-report
Closed

Fix docs dev server network isolation and MonaSans preload warnings#43725
pelikhan with Copilot wants to merge 5 commits into
mainfrom
copilot/multi-device-docs-testing-report

Conversation

Copilot AI commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

The multi-device docs tester reported a critical failure: the pre-agent step started astro preview on the runner host, but the agent's bash tool executes inside a Docker network sandbox — so localhost:4321 was unreachable from within the agent, causing every server check to fail silently. A secondary warning flagged repeated browser preload warnings for the MonaSans fonts.

Server startup: pre-agent → agent context

daily-multi-device-docs-tester.md

  • Removed the Start docs server and Wait for server readiness pre-agent steps (which ran on the host, outside the agent's network namespace)
  • Agent now starts the dev server itself in Step 1 using npm run dev, consistent with shared/docs-server-lifecycle.md:
nohup npm run dev -- --host 0.0.0.0 --port 4321 > /tmp/gh-aw/agent/preview.log 2>&1 &
  • Switched from astro preview (requires a prior build) to astro dev — no pre-built dist/ needed, and the dev server correctly serves all /gh-aw/* paths
  • Readiness polling now runs in the same context, so localhost:4321 resolves correctly

Font preload warnings

docs/src/components/CustomHead.astro

  • Removed <link rel="preload"> hints for MonaSans-Regular.woff2 and MonaSans-Bold.woff2
  • Fonts continue loading via @font-face in custom.css with font-display: swap; the preload hints were redundant and triggered browser warnings about unused preloads

pr-sous-chef run: https://github.com/github/gh-aw/actions/runs/28776499241

Generated by 👨‍🍳 PR Sous Chef · 7.38 AIC · ⌖ 10.2 AIC · ⊞ 5.6K ·
Comment /souschef to run again

Copilot AI and others added 2 commits July 6, 2026 07:26
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Move docs server startup from pre-agent steps into agent body using
  npm run dev per docs-server-lifecycle.md guidance. This ensures the
  server starts in the same network context as the agent's bash tool,
  making localhost:4321 accessible.
- Remove npm run build + astro preview pre-agent steps (not needed
  with dev server).
- Remove MonaSans font preload hints from CustomHead.astro that caused
  repeated browser preload warnings.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix multi-device testing report issues Fix docs dev server network isolation and MonaSans preload warnings Jul 6, 2026
Copilot AI requested a review from pelikhan July 6, 2026 07:34
@pelikhan
pelikhan marked this pull request as ready for review July 6, 2026 07:52
Copilot AI review requested due to automatic review settings July 6, 2026 07:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the multi-device docs tester’s docs server startup so it runs in the same (sandboxed) context as the agent’s checks, and removes redundant MonaSans font preload hints that were producing browser warnings. It also includes widespread, low-impact regeneration/normalization changes across generated *.lock.yml workflows (mostly trimming trailing newlines / block-scalar chomping).

Changes:

  • Move docs server startup + readiness polling into the daily-multi-device-docs-tester agent instructions (switching from astro preview to astro dev).
  • Remove MonaSans rel="preload" links from CustomHead.astro to stop unused-preload warnings.
  • Regenerate many workflow lockfiles with minor formatting/serialization adjustments.
Show a summary per file
File Description
docs/src/components/CustomHead.astro Removes MonaSans font preload tags to eliminate unused-preload warnings.
.github/workflows/daily-multi-device-docs-tester.md Updates tester instructions to start and poll an Astro dev server from the agent context.
.github/workflows/daily-multi-device-docs-tester.lock.yml Regenerated lockfile reflecting removal of host pre-agent server steps.
.github/workflows/workflow-skill-extractor.lock.yml Minor lockfile script serialization change (trailing newline normalization).
.github/workflows/workflow-health-manager.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/weekly-blog-post-writer.lock.yml Minor lockfile block-scalar chomping (`run:
.github/workflows/unbloat-docs.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/uk-ai-operational-resilience.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/test-quality-sentinel.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/step-name-alignment.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/static-analysis-report.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/stale-repo-identifier.lock.yml Minor lockfile block-scalar chomping (`run:
.github/workflows/spec-extractor.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/schema-consistency-checker.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/repository-quality-improver.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/release.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/prompt-clustering-analysis.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/pr-description-caveman.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/objective-impact-report.lock.yml Minor lockfile github-script serialization change (trailing newline normalization).
.github/workflows/mergefest.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/mattpocock-skills-reviewer.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/linter-miner.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/lint-monster.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/issue-arborist.lock.yml Minor lockfile block-scalar chomping (`run:
.github/workflows/impeccable-skills-reviewer.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/gpclean.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/glossary-maintainer.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/eslint-monster.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/docs-noob-tester.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/designer-drift-audit.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/design-decision-gate.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/dependabot-burner.lock.yml Minor lockfile github-script serialization change (trailing newline normalization).
.github/workflows/delight.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/dataflow-pr-discussion-dataset.lock.yml Minor lockfile block-scalar chomping (`run:
.github/workflows/daily-spdd-spec-planner.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/daily-security-observability.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/daily-community-attribution.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/daily-ambient-context-optimizer.lock.yml Minor lockfile github-script serialization change (trailing newline normalization).
.github/workflows/copilot-pr-nlp-analysis.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/copilot-centralization-drilldown.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/contribution-check.lock.yml Minor lockfile block-scalar chomping (`run:
.github/workflows/code-simplifier.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/cli-consistency-checker.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/ci-doctor.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/aw-failure-investigator.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/auto-triage-issues.lock.yml Minor lockfile block-scalar chomping (`run:
.github/workflows/architecture-guardian.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/agentic-token-trend-audit.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/agentic-token-optimizer.lock.yml Minor lockfile run string trailing newline normalization.
.github/workflows/agentic-token-audit.lock.yml Minor lockfile run string trailing newline normalization.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 50/50 changed files
  • Comments generated: 1
  • Review effort level: Low

Comment thread .github/workflows/daily-multi-device-docs-tester.md Outdated
@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address unresolved review comments, and rerun checks once the branch is up to date.

Generated by 👨‍🍳 PR Sous Chef · 7.38 AIC · ⌖ 10.2 AIC · ⊞ 5.6K ·
Comment /souschef to run again

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

PR Triage

Category chore
Risk Low
Score 35/100 (impact 15 + urgency 10 + quality 10)
Action defer

[DRAFT] Docs dev server fix: moves server startup inside agent context + removes redundant MonaSans preload hints. Still draft. Defer until author marks ready.

Run §28776188550

Generated by 🔧 PR Triage Agent · 98.5 AIC · ⌖ 11.4 AIC · ⊞ 5.5K ·

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
@pelikhan pelikhan closed this Jul 6, 2026
Copilot AI requested a review from gh-aw-bot July 6, 2026 08:23
Copilot stopped work on behalf of gh-aw-bot due to an error July 6, 2026 08:23
@github-actions
github-actions Bot deleted the copilot/multi-device-docs-testing-report branch July 21, 2026 02:59
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.

🔍 Multi-Device Docs Testing Report - 2026-07-06

4 participants