feat: enrich browser configs with context fields and merge browser card into CI/Env#262
Merged
Merged
Conversation
…se page The test-run-case summary showed CI / Env and Browser as two separate metadata cards. Merge them: CiEnvCard now takes an optional browser prop and renders a dense browser line (badge icon + screen resolution, with the full config in the badge tooltip), and TestCaseSummary drops the standalone Browser block and reflows the grid accordingly. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQnH7cNyDiC1BmPk3gqGdh
The dev-DB seeder split public/demo/seed.sql on every semicolon, which shattered any INSERT whose values contained a ';' (e.g. a browser user-agent string), dropping that row and cascading foreign-key failures. Split on statement boundaries instead, ignoring semicolons inside single-quoted string literals. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQnH7cNyDiC1BmPk3gqGdh
…a setup Give the demo browser configs the richer context fields (deviceScaleFactor, isMobile/hasTouch, locale, timezone, colorScheme, userAgent) so the merged CI / Env card's browser line and the badge tooltip have real content to show (e.g. Mobile Safari at 3x, a dark-mode 2x profile). Add an app:seed:dev script and an AGENTS.md section documenting how to run the app locally with sample data for verification and screenshots. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WQnH7cNyDiC1BmPk3gqGdh
Contributor
Coverage Report for Reporter (./reporter)
File CoverageNo changed files found. |
Contributor
Coverage Report for Application (./application)
File CoverageNo changed files found. |
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
2 participants
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.
What & why
This change enriches the demo seed data with realistic browser configuration context (device scale factor, mobile/touch flags, locale, timezone, color scheme, user agent) and refactors the test case summary UI to merge the standalone Browser card into the CI/Env card as a dense browser line.
Why:
BrowserBadge) and dense inline displayChanges:
generate-demo-seed.mjs: ExpandedBROWSER_CONFIGSwith full context fields (deviceScaleFactor, isMobile, hasTouch, locale, timezoneId, colorScheme, userAgent) for each of the 4 projects. Added aCHROME_UAconstant for reuse. Updated project comments to describe the testing scenario.TestCaseSummary.vue: Removed the standalone Browser card and merged it intoCiEnvCardby passing thebrowserprop. Updated grid layout logic to count CI/Env/Browser as a single column.CiEnvCard.vue: Added optionalbrowserprop and a dense browser line (icon + screen resolution) that displays when a browser is provided. Full browser config remains accessible via theBrowserBadgetooltip.seed-dev-from-demo.mjs: Fixed SQL statement splitting to correctly handle semicolons inside single-quoted string literals (e.g., user agents, commit messages). AddedsplitSqlStatements()function with proper quote-aware parsing.AGENTS.md: Updated documentation to reflect the merged card behavior and added a new "Running the app locally with sample data" section with step-by-step instructions for local verification with the dev DB.package.json: Addedapp:seed:devnpm script as an alias for the seed command.How was it tested?
Checklist
feat(...))AGENTS.mdwith local dev workflow)https://claude.ai/code/session_01WQnH7cNyDiC1BmPk3gqGdh