Skip to content

Conversation

@BaiyuScope3
Copy link
Collaborator

@BaiyuScope3 BaiyuScope3 commented Jan 14, 2026

Fix Testing UI and Security Updates

Summary

This PR fixes a JavaScript error in the testing UI and addresses a high-severity security vulnerability.

Changes

🐛 Bug Fix: Testing UI JavaScript Error

Fixed a syntax error in src/public/index.html that prevented the testing page from loading correctly:

Uncaught SyntaxError: Unexpected token 'else'
Uncaught ReferenceError: toggleAddAgent is not defined

Cause: Invalid } else { ... } else if { ... } structure in the format dimension parsing logic.

Fix: Restructured to proper nested conditionals:

if (format.dimensions) { ... }
else {
  if (dimensionAsset) { ... }
  else if (format.format_id.includes('x')) { ... }
}

🔒 Security: Update hono to 4.11.4

Updated hono from 4.11.3 to 4.11.4 to address high-severity vulnerabilities:
GHSA-3vhc-576x-3qv4 - JWK Auth Middleware JWT algorithm confusion
GHSA-f67f-6cw9-8mq4 - JWT Middleware algorithm confusion via unsafe HS256 default

🔄 Chore: Regenerate Types

Regenerated TypeScript types to sync with v2.6 schemas.

Testing

[x] Local server starts without errors
[x] Testing UI loads correctly
[x] No JavaScript console errors
[x] npm audit shows 0 vulnerabilities

@BaiyuScope3 BaiyuScope3 marked this pull request as ready for review January 14, 2026 16:40
@BaiyuScope3 BaiyuScope3 merged commit c1d72d0 into main Jan 14, 2026
11 checks passed
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