GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix: correct scanner output, report timestamps, and web navigation by Dumbris · Pull Request #1830 · smart-mcp-proxy/mcpproxy-go

Conversation

@Dumbris

@Dumbris Dumbris commented Nov 6, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

Description

Fix four P2 defects found in the Web UI, CLI and scanner UX audit:

  • Synchronous single-server and batch scans honor JSON/YAML output. Progress stays on stderr, and an unavailable report fails instead of printing a text success hint into machine-readable stdout.
  • Scan reports retain the latest persisted scan/completion time across reads, including merged companion passes with missing report dates. Reading a report no longer makes an old scan appear fresh.
  • The Tools refresh control has the accessible name “Refresh tools”.
  • Mobile navigation closes the drawer after successful navigation or selecting the active link, while aborted navigation keeps it open and desktop collapse state remains independent.

Regression tests cover structured output, intermediate batch progress, report errors, stored timestamps, companion passes and navigation outcomes. Updated App test fixtures provide a memory router.

Testing

  • Production frontend typecheck/build and embedded daemon build
  • Full frontend suite: 2,795 tests passed
  • Full scanner package suite passed
  • Focused Go race suite: 42 tests/subtests passed
  • Focused Go lint: 0 issues
  • Live isolated CLI/REST/MCP rerun: 14 cases passed
  • Browser rerun: 5 cases passed; refresh works with mouse/Enter; mobile overlay becomes hidden; report time survives reload; no uncaught errors during authenticated checks
  • Hard-tier and review-tier scanner corpus gates passed; warning/dangerous fixtures remain quarantined and MCP calls are blocked
  • Independent review: CLEAN
  • Required repository CI passed on the current head
  • Current-head rerun: 42 Go race checks and 24 UI checks passed; application code is unchanged from the full QA run
  • Optional shuffled and end-to-end suites (still running)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@Dumbris
Fix scanner output, scan timestamps, and web UX defects
2dd2161
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 6, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Deploying mcpproxy-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: bd5ff43
Status: ✅  Deploy successful!
Preview URL: https://d349261e.mcpproxy-docs.pages.dev
Branch Preview URL: https://codex-fix-qa-ux-defects.mcpproxy-docs.pages.dev

View logs

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@github-actions

github-actions Bot commented Nov 6, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

📦 Build Artifacts

Workflow Run: View Run
Branch: codex/fix-qa-ux-defects

Available Artifacts

  • archive-darwin-amd64 (32 MB)
  • archive-darwin-arm64 (29 MB)
  • archive-linux-amd64 (20 MB)
  • archive-linux-arm64 (18 MB)
  • archive-windows-amd64 (32 MB)
  • archive-windows-arm64 (28 MB)
  • frontend-dist-pr (0 MB)
  • installer-dmg-darwin-amd64 (28 MB)
  • installer-dmg-darwin-arm64 (25 MB)
  • smart-mcp-proxymcpproxy-goMPIZYT.dockerbuild (0 MB)

How to Download

Option 1: GitHub Web UI (easiest)

  1. Go to the workflow run page linked above
  2. Scroll to the bottom "Artifacts" section
  3. Click on the artifact you want to download

Option 2: GitHub CLI

gh run download 42349004619 --repo smart-mcp-proxy/mcpproxy-go

Note: Artifacts expire in 14 days.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@codecov-commenter

codecov-commenter commented Nov 6, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 87.09677% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cmd/mcpproxy/security_cmd.go 78.94% 2 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@Dumbris
Remove QA evidence artifacts from fixes PR
bd5ff43
@Dumbris
Dumbris marked this pull request as ready for review November 7, 2026 06:56
@Dumbris
Dumbris enabled auto-merge (squash) November 7, 2026 07:02
@Dumbris
Dumbris merged commit 2d12411 into main Nov 7, 2026
52 checks passed

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@Dumbris @codecov-commenter
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.