feat(browser): add CDP support and reverse ATS discovery scanner#3
Open
SparshGarg999 wants to merge 1 commit into
Open
feat(browser): add CDP support and reverse ATS discovery scanner#3SparshGarg999 wants to merge 1 commit into
SparshGarg999 wants to merge 1 commit into
Conversation
|
Welcome to career-ops, @SparshGarg999! Thanks for your first PR. A few things to know:
We'll review your PR soon. Join our Discord if you have questions. |
SparshGarg999
pushed a commit
that referenced
this pull request
Jul 4, 2026
* feat(web): cost affordance at the point of paid action (P1.6) UX audit P1.6 (CPO priority #3 — tokens are the community's #1 measurable pain; surprise-cost = churn). The paid actions offered no cost cue at the trigger. Surfaces the existing CostBadge (spend = brand orange, not red — a chosen evaluation is the valuable action, not a hazard) on the roomy single-action surfaces, mirroring Explore's token-honesty: - GeneratePdfButton (component-level → shows in the report viewer): a 'Uses tokens' badge beside 'Generate tailored CV' in its not-yet-generated state; - QuickEvaluate (home): a one-line cue + badge under the paste field — 'Evaluation runs on your own AI — your key, your machine.' The dense pipeline-row Evaluate button is deliberately left for P1.4 (row hierarchy), where a per-row badge would fight the noise-reduction goal. Verified E2E: report santifer#87 (no PDF) shows the badge; home in-between shows the cue; typecheck clean. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * polish(web): cost 'spend' badge is muted, not brand-orange (design-review) career-ops-ux system correction on santifer#1536: orange is reserved for the primary action (incl. 'Run your first FREE scan'), so an orange 'Uses tokens' badge collides semantically ('go/free' vs 'costs') AND — now the button is neutral — makes the warning shout louder than the action. Aligns to the color-semantics lock (-> DESIGN_SYSTEM.md): GREEN = free/celebrate, NEUTRAL/muted (+ coin) = spend/inform, ORANGE = primary action (reserved). CostBadge is the shared primitive, so this one change propagates to every cost cue — Explore, the report's Generate-CV, and P1.4's inbox legend — as a single pattern. Muted AA-verified: 5.68 light / 8.34 dark. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Purpose
Addresses issue santifer#727: Integrates Chromium DevTools Protocol (CDP) support for liveness check, scanning, and PDF generation.
Changes
waitUntil: 'load') and inlined local fonts asdata:URLs to prevent Chromium block onfile://protocols.--rediscover-404moved-role search option.CDP_URLenvironment variable, options, and commands.