diff --git a/docs/audits/evidence/career-001/README.md b/docs/audits/evidence/career-001/README.md new file mode 100644 index 0000000..865b457 --- /dev/null +++ b/docs/audits/evidence/career-001/README.md @@ -0,0 +1,13 @@ +# CAREER-001 evidence index + +Updated: 2026-08-09 + +No screenshot is claimed for this package because the browser session had already been finalized. + +- Execution-path and design evidence: `docs/verification/career-001-career-workspace.md` +- Command log: `docs/audits/verification-log.md` V-117–V-119 +- State/navigation tests: `job-tracker-ui/src/career-workspace-page.test.tsx` +- Approval-gate and profile regression tests: `job-tracker-ui/src/profile-page.test.tsx` +- Implementation: `job-tracker-ui/src/views/career/CareerWorkspaceOverview.tsx`, `CareerProfilePage.tsx`, `CareerWorkspacePage.tsx`, `ProfileCompleteness.tsx` +- Implementation commit: `268b3a0` +- Synthetic data only; no private CV, provider, model, production service or external request was used. diff --git a/docs/audits/verification-log.md b/docs/audits/verification-log.md index 7d7f9d2..8ed3414 100644 --- a/docs/audits/verification-log.md +++ b/docs/audits/verification-log.md @@ -148,3 +148,6 @@ Output was reduced to filenames and commit counts. The token artifact appears un | V-114 | Complete trace of import HTML cleanup/language/tagging, deterministic matcher, analysis endpoint/UI, match endpoint/UI, learning-item sync and storage/cache behavior | Repository root | Reproduce QA-001 low-information term path and version implications | PASS — visible terms come from English-only single-token ranking in `JobCvMatchService`; imported HTML is cleaned earlier but manual text may not be; result is on-demand and has no historical analysis cache; AI is not involved | Confirmed source execution path | Application-related | | V-115 | `dotnet test ... --filter "...JobCvMatchServiceTests|...ApplicationIntelligenceTests|...JobApplicationsEndpointBehaviorTests|...ApplicationChecklist"`; `npm.cmd test ... match-score-panel.test.tsx application-intelligence.test.tsx`; final matcher/UI focused rerun | Repository root / `job-tracker-ui` | Verify seven required fixtures, affected endpoints/learning lifecycle and honest label | PASS — affected backend 54/54; affected UI 13/13; final matcher 15/15 and label 3/3 | Synthetic job/CV text only; no model/provider/browser | N/A | | V-116 | Full `dotnet test ... --no-restore`; full `npm.cmd test -- --runInBand --forceExit`; `npm.cmd run build`; `git diff --check` | Repository root / `job-tracker-ui` | QA-001 full regression, TypeScript/build and patch hygiene | PASS — backend 601/601; frontend 48/48 suites and 172/172 tests; production build; no whitespace errors | Jest existing force-exit/open-handle notice; browser/production presentation not run | Environmental/deployment limitation | +| V-117 | Bounded `rg`/`Get-Content` of Career Workspace/Profile/Builder components, CV variant client, application CV assets, routes, requirements and tests | Repository root | Revalidate CAREER-001 state ownership, navigation, approval gate and existing job-specific CV boundary | PASS — profile component owns completeness/import state; variants carry optional job ownership; Apply/Discard is the only extraction merge path; old wrapper lacked resumable/recent/first-run hierarchy | No browser or production execution; behavior labels distinguish source inspection | Confirmed product hierarchy gap | +| V-118 | `npm.cmd test -- --runInBand src/career-workspace-page.test.tsx src/profile-page.test.tsx` | `job-tracker-ui` | Verify first/returning/incomplete/loading/processing/review/failure/recent-document states and unchanged approval gate | PASS — 2 suites, 16/16 tests | Initial assertions exposed intentional duplicate labels; duplicate page controls/completeness were removed and selectors corrected without weakening behavior | Implementation/test refinement resolved | +| V-119 | `npm.cmd test -- --runInBand`; `npm.cmd run build`; `git diff --check` | `job-tracker-ui` / repository root | CAREER-001 full regression, TypeScript/build and patch hygiene | PASS — 49/49 suites, 178/178 tests; optimized production build; no whitespace errors | Repository line-ending notices only; browser/production checks not run | Environmental/deployment limitation | diff --git a/docs/verification/application-action-matrix.md b/docs/verification/application-action-matrix.md index 225841b..149d0ec 100644 --- a/docs/verification/application-action-matrix.md +++ b/docs/verification/application-action-matrix.md @@ -46,5 +46,6 @@ This is the rolling action-level evidence index. `PASS (automated/runtime)` is n | Authentication UI | unified username/password and provider alternatives; invalid/cancel/return behavior | PASS (components; mocked providers) | PARTIAL — local dark-theme form at 375/768/1440 | NOT RUN | `ux-001-unified-authentication.md` | | Theme state | Light/Dark/System precedence, login/logout scope, refresh/navigation and two-tab synchronization | PASS (state/provider/bootstrap tests) | PASS/PARTIAL — anonymous local browser at 375/768/1440 | NOT RUN | `ux-002-deterministic-theme-state.md` | | Job match terms | bilingual/noisy/short/technology-heavy/filler extraction and honest labels | PASS (seven fixtures + component tests) | NOT RUN | NOT RUN | `qa-001-job-term-quality.md` | +| Career Workspace | first/returning/incomplete/loading/import processing/review/failure and recent CV actions | PASS (components; approval gate regression) | NOT RUN | NOT RUN | `career-001-career-workspace.md` | Remaining product actions are `NOT STARTED` in the master plan and will be added as their work packages enter verification. The in-app browser is available for local UI checks; configured/real-provider and production access are not documented/configured. diff --git a/docs/verification/career-001-career-workspace.md b/docs/verification/career-001-career-workspace.md new file mode 100644 index 0000000..0b40eb8 --- /dev/null +++ b/docs/verification/career-001-career-workspace.md @@ -0,0 +1,49 @@ +# CAREER-001 Career Workspace redesign + +Updated: 2026-08-09 + +Status: `IMPLEMENTED — NOT VERIFIED`. State-focused component tests, the full frontend suite and the production build pass. Browser and production checks remain. + +## Revalidated execution path + +- `/career` renders `CareerWorkspacePage`, which delegates the master-profile state to `CareerProfilePage`. +- `CareerProfilePage` loads `/career/profile`, `/auth/me`, `/profile-cv/runs` and version history. Active durable operations are polled; pending-review diffs are loaded separately. +- CV import/rebuild/improve/reprocess create durable processing runs. Extracted changes are not written into the career profile until the user selects **Apply changes**; **Discard** remains available. +- The previous wrapper presented a duplicate page heading, a large explanatory alert and one CV Builder action. Recent CV documents, first-run guidance, missing information and resumable import states were not visible at workspace level. +- General documents live at `/career/builder`; job-specific selection/attachment lives in a saved job workspace. CV variants already carry optional `jobApplicationId` ownership metadata. + +## Implemented contract + +- The workspace starts with one `h1`, a short next-step prompt and an explicit CV Builder action. The requested long explanatory paragraph was removed. +- Four responsive action cards lead to career-profile editing, CV import/review, general CV creation and the saved-job path for job-specific CVs. +- Completeness and up to three missing areas are visible before the editor. The former duplicate completeness panel is hidden on this route while version restore remains available. +- Durable import state is summarized as import, processing, review or failure and links to the existing run history/actions. +- First-run, profile-load error, recent-CV loading/error/empty and returning-user states are explicit. Recent CVs are sorted by update time and show job-specific context. +- Anchor targets use scroll margins; action navigation is native keyboard-focusable links. The grid collapses at small widths. +- Existing profile persistence, extraction diff, low-confidence selection, Apply/Discard approval and version restore behavior are unchanged. + +## Verification + +- Focused Career Workspace and Career Profile: 2 suites, 16/16 tests. +- State coverage includes first-run, returning/incomplete profile, recent general/job-specific CVs, queued processing, pending review, failure, loading and load error. +- Existing pending-review test proves changes are applied only after the explicit accept request. +- Full frontend: 49/49 suites, 178/178 tests. +- Production frontend build and TypeScript: pass. +- `git diff --check`: pass apart from repository line-ending notices. + +## Remaining gates + +- Browser tooling was finalized earlier in this session, so CAREER-001 was not inspected in a running browser. Required 375/768/1440, Light/Dark, keyboard/focus, Norwegian and rendered error/processing checks remain. +- Synthetic-account production smoke and deployed route/anchor behavior remain unavailable without production access. +- The saved-job path explains where a job-specific CV is managed; the deeper CV Builder/application interaction redesign remains CAREER-002/JOBS-001 scope. + +## Evidence + +- Evidence index: `docs/audits/evidence/career-001/README.md` +- Commands/results: `docs/audits/verification-log.md` V-117–V-119 +- Focused tests: `job-tracker-ui/src/career-workspace-page.test.tsx`, `job-tracker-ui/src/profile-page.test.tsx` +- Implementation commit: `268b3a0` + +## Rollback + +Revert `268b3a0`. No schema, dependency, configuration, provider or stored profile/CV data changes are involved. diff --git a/docs/work-programmes/decisions.md b/docs/work-programmes/decisions.md index 065cc8f..6042fc6 100644 --- a/docs/work-programmes/decisions.md +++ b/docs/work-programmes/decisions.md @@ -339,3 +339,13 @@ - **Consequences:** term changes appear on the next request; obsolete pending generated learning items auto-complete under existing behavior while done/dismissed decisions remain. The curated vocabulary remains intentionally bounded and test-driven. - **User approval required:** No; this is the deterministic-first implementation explicitly required by both programmes. - **Reversible:** Yes. Revert `da1aa8b`; no schema, cache or provider state changes. + +## DEC-035 — Career Workspace derives actions from existing domain state + +- **Date:** 2026-08-09 +- **Decision:** Render one action-oriented overview from the completeness and durable import state already owned by `CareerProfilePage`; load only the separate recent-CV list. Link actions to the existing profile/import anchors, builder and saved-job workflow, and retain extraction Apply/Discard as the sole merge gate. +- **Reason/evidence:** duplicating `/career/profile` and `/profile-cv/runs` requests in the route wrapper would create competing loading/polling state. The existing page already has correct durable run, review and version data, while CV variants are a separate bounded list. The old wrapper duplicated headings and explanation without exposing next actions. +- **Alternatives considered:** a second workspace data loader; moving all profile state into a new global store; direct one-click job-CV creation without a job choice; rewriting the profile editor. These add inconsistent state, premature architecture or bypass the established job/application boundary. +- **Consequences:** the page presents concise state-aware navigation without changing API, profile persistence or imported-data approval semantics. Browser and production verification remain before `DONE`; deeper builder/job interaction remains in CAREER-002/JOBS-001. +- **User approval required:** No; this is the smallest cohesive implementation of the approved programme requirement. +- **Reversible:** Yes. Revert the CAREER-001 implementation commit; no data/configuration migration exists. diff --git a/docs/work-programmes/master-progress.md b/docs/work-programmes/master-progress.md index 6753189..26a9e5b 100644 --- a/docs/work-programmes/master-progress.md +++ b/docs/work-programmes/master-progress.md @@ -2,17 +2,17 @@ Updated: 2026-08-09 -- **Overall programme status:** Active. Six packages are locally verified; seventeen packages through QA-001 are implemented with automated/runtime evidence but blocked from applicable browser/provider/production gates; CAREER-001 is in progress. -- **Current work package:** `CAREER-001` — Career Workspace action-oriented redesign (`IN PROGRESS`); requirements and current first/returning/incomplete/processing/error hierarchy are the next trace. +- **Overall programme status:** Active. Six packages are locally verified; eighteen packages through CAREER-001 are implemented with automated/runtime evidence but blocked from applicable browser/provider/production gates; CAREER-002 is in progress. +- **Current work package:** `CAREER-002` — CV Builder interaction redesign (`IN PROGRESS`); revalidate existing editor/list behavior before changing the already substantial implementation. - **Completed work packages:** None are `DONE`; all repository security/AI packages still have applicable browser, provider and/or production gates. - **Locally verified work:** SEC-001, SEC-002, SEC-003, SEC-005A, CORE-001 and PROD-002 (`VERIFIED LOCALLY`). -- **Implemented, verification incomplete:** SEC-004, SEC-005B, SEC-008, CORE-002, BG-001, OPS-001A/B/C, POL-001/002, AI-001/002/003/004, UX-001/002 and QA-001 (`IMPLEMENTED — NOT VERIFIED`). QA-001 now cleans HTML/chrome, filters bilingual filler, preserves curated punctuation/phrases and uses honest labels; all automated/build gates pass. +- **Implemented, verification incomplete:** SEC-004, SEC-005B, SEC-008, CORE-002, BG-001, OPS-001A/B/C, POL-001/002, AI-001/002/003/004, UX-001/002, QA-001 and CAREER-001 (`IMPLEMENTED — NOT VERIFIED`). CAREER-001 now exposes first-run/profile/import/review/failure/general/job-specific/recent-CV actions while preserving explicit Apply/Discard; all automated/build gates pass. - **Production-verified work:** None. - **Blocked work:** SEC-006 requires explicit internet/package-index permission; PROD-001/003/004 and REL-001 require documented production access and unfinished dependencies. Real provider, SMTP/MariaDB and production environments are unavailable; the in-app browser is available for local UI checks. - **Deferred work:** None. Conditional multi-replica coordination, model deletion, realtime operation delivery and unrelated production changes remain outside current packages. -- **Next five work packages:** CAREER-001 Career Workspace redesign; CAREER-002 CV Builder redesign; EMAIL-001 consolidated email experience; JOBS-001 job-search/application redesign; HOME-001 homepage/Pro promotion. SEC-006/007 resume after package-index permission. -- **Status counts:** 6 `VERIFIED LOCALLY`; 17 `IMPLEMENTED — NOT VERIFIED`; 1 `IN PROGRESS`; 10 `NOT STARTED`; 5 `BLOCKED`; 0 `DONE`; 0 `DEFERRED`. -- **Test status:** backend 601/601; QA-001 focused matcher 15/15 and affected backend 54/54; frontend 48/48 suites and 172/172 tests, affected UI 13/13; production build and patch check pass. QA-001 browser/production presentation was not run; prior UX-002 browser and sidecar 22/22 remain green. +- **Next five work packages:** CAREER-002 CV Builder redesign; EMAIL-001 consolidated email experience; JOBS-001 job-search/application redesign; HOME-001 homepage/Pro promotion; UX-003 authentication/profile polish. SEC-006/007 resume after package-index permission. +- **Status counts:** 6 `VERIFIED LOCALLY`; 18 `IMPLEMENTED — NOT VERIFIED`; 1 `IN PROGRESS`; 9 `NOT STARTED`; 5 `BLOCKED`; 0 `DONE`; 0 `DEFERRED`. +- **Test status:** backend 601/601; frontend 49/49 suites and 178/178 tests; CAREER-001 focused 16/16; production build and patch check pass. CAREER-001 browser/production presentation was not run because browser tooling was finalized; prior UX-002 browser and sidecar 22/22 remain green. - **Deployment status:** No deployment performed. No production migrations were run. AI operation worker remains disabled by default. - **Production status:** Unchanged and unverified. No provider/model call, model pull, external request or paid API occurred. - **Known regressions:** None found by automated suites. Jest still needs `--forceExit` and reports its existing open-handle notice. Direct clean EF-only SQLite migration still hits the pre-existing historical blank-chain defect before later migrations; normal startup owns reconciliation. Cross-feature monthly AI usage accounting remains a rollout gap. @@ -44,5 +44,6 @@ Updated: 2026-08-09 - `docs/verification/ux-001-unified-authentication.md` - `docs/verification/ux-002-deterministic-theme-state.md` - `docs/verification/qa-001-job-term-quality.md` +- `docs/verification/career-001-career-workspace.md` - `docs/verification/prod-002-ai-evaluation.md` - `docs/work-programmes/master-work-plan.md` diff --git a/docs/work-programmes/master-work-plan.md b/docs/work-programmes/master-work-plan.md index f111c6a..460f6a2 100644 --- a/docs/work-programmes/master-work-plan.md +++ b/docs/work-programmes/master-work-plan.md @@ -16,7 +16,7 @@ Allowed statuses are `NOT STARTED`, `IN PROGRESS`, `IMPLEMENTED — NOT VERIFIED `DONE` requires every applicable acceptance criterion, focused and regression tests, browser/accessibility/theme/mobile checks, tenant and entitlement checks, documentation, migration/rollback evidence, and production verification. Repository-only work that still requires production is at most `VERIFIED LOCALLY`. -Exactly one implementation item may be `IN PROGRESS`. As of this revision it is **CAREER-001**. +Exactly one implementation item may be `IN PROGRESS`. As of this revision it is **CAREER-002**. ## Consolidated dependency order @@ -613,11 +613,11 @@ Ordering differences from the suggested list: - **Required tests:** first/returning/incomplete/processing/failure state and navigation. - **Required browser verification:** three widths, light/dark, keyboard/focus/loading/empty/error/Norwegian. - **Required production verification:** synthetic account smoke. -- **Status:** `IN PROGRESS`. -- **Blocker:** browser tooling must be available for completion. -- **Evidence:** source requirements and current Career Workspace state hierarchy are next for complete trace; CORE-001 and AI-004 contracts are available. -- **Commit:** none. -- **Remaining work:** no master-profile overwrite. +- **Status:** `IMPLEMENTED — NOT VERIFIED`. +- **Blocker:** browser session was already finalized; three-width/theme/keyboard/Norwegian checks and production synthetic-account smoke remain. +- **Evidence:** `docs/verification/career-001-career-workspace.md`; V-117–V-119; focused 16/16, full frontend 49/49 suites and 178/178 tests, production build. State-aware actions/recent CVs are implemented and the Apply/Discard gate is unchanged. +- **Commit:** `268b3a0` (`feat(career): clarify workspace actions`). +- **Remaining work:** browser and production gates only; deeper builder/job-specific interaction belongs to CAREER-002/JOBS-001. ### CAREER-002 — CV Builder interaction redesign and external research @@ -631,9 +631,9 @@ Ordering differences from the suggested list: - **Required tests:** editing/collapse/add/delete/reorder/save/failure/persistence/preview. - **Required browser verification:** authorized FlowCV research if accessible, never bypass auth; original JobTracker design at three widths/themes/keyboard/focus. - **Required production verification:** existing variants/edit/export/public render smoke. -- **Status:** `NOT STARTED`. +- **Status:** `IN PROGRESS`. - **Blocker:** FlowCV may require manual authenticated session; implementation can proceed from local evidence if research is labeled blocked. -- **Evidence:** existing builder is more complete than the programme's premise; redesign must begin with a real gap analysis. +- **Evidence:** CAREER-001 is automated/build verified; existing builder is more complete than the programme's premise. Complete gap analysis and safe FlowCV access check are next. - **Commit:** none. - **Remaining work:** avoid rewriting already-working features. diff --git a/docs/work-programmes/session-handoff.md b/docs/work-programmes/session-handoff.md index 05a48b3..4f85b5b 100644 --- a/docs/work-programmes/session-handoff.md +++ b/docs/work-programmes/session-handoff.md @@ -2,17 +2,17 @@ Updated: 2026-08-09 -- **Exact current task:** CAREER-001 — re-read Career Workspace requirements and trace first-run, completeness, CV import review, queued processing/failure and recent-document actions before editing. -- **Last completed step:** QA-001 implementation `da1aa8b` was committed. Bilingual/chrome filtering, bounded phrases, expanded canonical technology tags and honest important-term labels pass all automated/build gates. Evidence/tracking are ready for their own commit and push. -- **Files currently modified:** QA-001 verification/evidence/tracking documents only. QA-001 application code is committed; no screenshots were captured. -- **Commands already run:** QA-001 complete import/language/tagger/matcher/API/UI/learning/cache trace; seven required fixtures; affected and full backend/frontend tests; production build; diff checks; implementation commit. See V-114–V-116. -- **Test results:** backend 601/601; QA matcher 15/15; affected backend 54/54; frontend 48/48 suites and 172/172 tests; affected UI 13/13; production build and diff check pass. Browser/production QA presentation remains. -- **Services currently running:** none started by this session. The Next development server was stopped and in-app browser tabs finalized. Pre-existing Docker services were not changed. -- **Temporary files or processes:** none. No dependency declaration, database or private artifact changed. -- **Production changes currently active:** none. No deployment, provider/model call, private CV access, model pull, paid service, production migration or external payload occurred. -- **Rollback status:** QA-001 has no schema/cache/dependency migration; revert `da1aa8b` to restore the prior term set/labels. Earlier rollback records remain current. Production is unchanged. -- **Uncommitted changes:** QA-001 evidence/master tracking documents only; commit/push these before CAREER-001 application code. -- **Known failures:** QA-001 browser/production presentation; authenticated two-user/production theme smoke; configured/real-provider UX-001; MariaDB/SMTP/production unavailable; SEC-006 needs explicit internet/package-index permission; parser isolation remains SEC-007; historical clean EF-only SQLite chain needs startup reconciliation; Jest retains its open-handle notice. -- **Exact next action:** commit and push QA-001 evidence, then read `work.md:265-307`, Career Workspace components/tests, CORE-001 data shapes and AI-004 operation states; reproduce first/returning/incomplete/queued/review/failure paths before editing. -- **Work that can continue independently:** CAREER-001/002 and later repository UX packages. SEC-006/007 await package-index permission; PROD-001/003/004 await production access. -- **Decisions still required from the user:** none for synthetic repository work. Internet/package upgrades, private CV access, external/paid providers and production actions retain their explicit approval/safety gates; SEC-009 retention/legal policy remains unresolved. +- **Exact current task:** CAREER-002 — re-read the CV Builder redesign requirements and inventory existing list/editor/content/reorder/hide/save/preview/export/version/responsive/accessibility behavior before editing. +- **Last completed step:** CAREER-001 implementation was committed as `268b3a0`. The action-oriented workspace passes 16/16 focused tests, the full frontend passes 49 suites/178 tests, and the production build passes. Evidence/tracking are ready for their own commit and push. +- **Files currently modified:** CAREER-001 verification, evidence and master tracking documents only. Application code/tests are committed. No application data, dependency, schema or configuration changed. +- **Commands already run:** bounded Career/Profile/Builder/application-assets trace; focused Jest twice after refinement; full frontend Jest; production frontend build; diff/stat/check. See V-117–V-119. +- **Test results:** CAREER-001 focused 16/16; frontend full 49/49 suites and 178/178 tests; production build/TypeScript and patch check pass. Browser and production checks remain. +- **Services currently running:** none. The earlier Next development server was stopped and browser tabs finalized. Pre-existing Docker services were not changed. +- **Temporary files or processes:** none. No private CV, provider, model, paid service or external request was used. +- **Production changes currently active:** none. No deployment, migration, provider call or production payload occurred. +- **Rollback status:** CAREER-001 has no schema/config/dependency migration. Revert `268b3a0`. Earlier rollback records remain current; production is unchanged. +- **Uncommitted changes:** CAREER-001 evidence/tracking documents only, pending their reviewed commit and push. +- **Known failures:** CAREER-001 browser widths/themes/keyboard/Norwegian and production synthetic smoke; QA-001 browser/production; real-provider authentication; MariaDB/SMTP/production unavailable; SEC-006 needs explicit internet/package-index permission; parser isolation remains SEC-007; historical clean EF-only SQLite chain needs startup reconciliation; Jest has an existing open-handle issue only when forced exit is used. +- **Exact next action:** commit/push CAREER-001 evidence, then read `work.md:388-452`, `CvBuilderPage.tsx`, `CvBuilderEditor.tsx`, related tests and backend variant/render/export endpoints; write a gap matrix before the smallest CAREER-002 change. +- **Work that can continue independently:** CAREER-002 and later repository UX packages. SEC-006/007 await package-index permission; PROD-001/003/004 await production access. +- **Decisions still required from the user:** none for synthetic repository work. Internet/package upgrades, authenticated FlowCV/private CV access, external/paid providers and production actions retain explicit approval/safety gates; SEC-009 retention/legal policy remains unresolved.