Repository navigation
Add pinned Modern Web Guidance to frontend build and review - #15
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Refs #14. Frontend tasks now consult pinned Modern Web Guidance before choosing browser patterns. Web reviews scan the affected local frontend paths and verify candidates against project browser policies. The adapter uses upstream 1.0.0 (Apache-2.0), forces telemetry and file logging off, and refuses mutation commands, outside paths and descendant symlinks.
The skill and its build/review resources remain independently installable. The catalogue records upstream source, commit, licence and version; astack, web-design, dogfood and ship carry portable resources. Version 0.12.0 includes a confirmation-flow example and synthetic routing/consumer evidence. No product code, browser targets, account behavior or production schedules change.
Validation:
bun run checkpassed (54 package/install cases, packed CLI installation for both clients, all script help journeys, new runner consumer contract). Final fresh routing passed all 50 existing direct, 27 near-miss and six new cases. Initial over-selection and limitations are retained inevals/results/modern-web-2026-11-14/. Both baseline and candidate confirmation plans preserved the maintained library and Safari 17.4; this is guidance-use evidence, not measured improvement in shipped UI or speed.Independent critique reproduced a nested source-symlink escape and inherited file logging. Both were fixed, regression-covered and independently replayed successfully. Actual pinned search/retrieval and a local synthetic scan ran; the scan's zero findings is not a quality claim. No real deletion, authenticated session or product modernization was exercised.
Barbossa's matching selection/prompt change is PR galleonlabs/barbossa-company-agent#272, with a separate drained self-release receipt. This issue remains open until that runtime rollout is read back; this active child will not restart its supervisor.