qfai 1.7.13 → 1.7.14

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (43) hide show
  1. package/README.md +7 -5
  2. package/assets/init/.qfai/assistant/agents/frontend-engineer.md +2 -2
  3. package/assets/init/.qfai/assistant/agents/product-experience-architect.md +2 -2
  4. package/assets/init/.qfai/assistant/agents/product-surface-reviewer.md +1 -1
  5. package/assets/init/.qfai/assistant/skills/qfai-discussion/SKILL.md +44 -18
  6. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/01_Context.md +9 -0
  7. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/03_Story-Workshop.md +1 -1
  8. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/04_Sources.md +6 -6
  9. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/14_Review-Request.md +7 -7
  10. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/review/Rxx_reviewer.md +2 -2
  11. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/review/review_request.md +2 -2
  12. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/10_implementation_strategy.md +31 -13
  13. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/20_trend_scan.md +41 -0
  14. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/23_design_eval_aggregate.md +12 -0
  15. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/40_screen_contracts.md +1 -1
  16. package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/50_review_input_bundle.md +2 -0
  17. package/assets/init/.qfai/assistant/skills/qfai-implement/SKILL.md +6 -2
  18. package/assets/init/.qfai/assistant/skills/qfai-prototyping/SKILL.md +264 -34
  19. package/assets/init/.qfai/assistant/steering/agent-catalog.yml +1 -1
  20. package/assets/init/.qfai/assistant/steering/agent-routing.yml +1 -1
  21. package/assets/init/.qfai/assistant/steering/manifest.md +4 -7
  22. package/assets/init/.qfai/assistant/steering/product.md +6 -6
  23. package/assets/init/.qfai/assistant/steering/review-profiles.yml +3 -0
  24. package/assets/init/.qfai/assistant/steering/ui-definition-protocol.md +2 -2
  25. package/assets/init/.qfai/contracts/ui/README.md +2 -2
  26. package/assets/init/.qfai/discussion/README.md +14 -22
  27. package/assets/init/.qfai/evidence/README.md +21 -12
  28. package/assets/uix-rev/comparison-review.md +3 -15
  29. package/assets/uix-rev/contracts-review.md +5 -2
  30. package/assets/uix-rev/scoring-review.md +10 -2
  31. package/assets/uix-rev/strategy-review.md +11 -7
  32. package/dist/cli/index.cjs +1993 -1279
  33. package/dist/cli/index.cjs.map +1 -1
  34. package/dist/cli/index.mjs +1930 -1216
  35. package/dist/cli/index.mjs.map +1 -1
  36. package/dist/index.cjs +1989 -1269
  37. package/dist/index.cjs.map +1 -1
  38. package/dist/index.d.cts +75 -62
  39. package/dist/index.d.ts +75 -62
  40. package/dist/index.mjs +1926 -1207
  41. package/dist/index.mjs.map +1 -1
  42. package/package.json +1 -1
  43. package/assets/uix-rev/migration-review.md +0 -17
@@ -38,14 +38,23 @@ Optional:
38
38
 
39
39
  ## Obligation matrix
40
40
 
41
- | surface / mode | specs | runtimeGate | uiFidelity | render evidence | browser QA | fullHarness |
42
- | ------------------------- | -------- | ----------- | ----------------------------- | --------------- | ---------- | ----------- |
43
- | non-ui / low-cost | required | optional | n/a | n/a | n/a | absent |
44
- | non-ui / standard | required | optional | n/a | n/a | n/a | absent |
45
- | non-ui / full-harness | required | optional | n/a | n/a | n/a | required |
46
- | ui-bearing / low-cost | required | optional | optional (`skeleton` allowed) | optional | optional | absent |
47
- | ui-bearing / standard | required | optional | required | optional | optional | absent |
48
- | ui-bearing / full-harness | required | required | required | required | required | required |
41
+ | surface / mode | specs | runtimeGate | uiFidelity | render evidence | browser QA | fullHarness |
42
+ | ---------------------- | -------- | ----------- | ----------------------------- | --------------- | ---------- | ----------- |
43
+ | web / low-cost | required | optional | optional (`skeleton` allowed) | optional | optional | absent |
44
+ | web / standard | required | optional | required | optional | optional | absent |
45
+ | web / full-harness | required | required | required | required | required | required |
46
+ | mobile / low-cost | required | optional | optional (`skeleton` allowed) | optional | optional | absent |
47
+ | mobile / standard | required | optional | required | optional | optional | absent |
48
+ | mobile / full-harness | required | required | required | required | required | required |
49
+ | desktop / low-cost | required | optional | optional (`skeleton` allowed) | optional | optional | absent |
50
+ | desktop / standard | required | optional | required | optional | optional | absent |
51
+ | desktop / full-harness | required | required | required | required | required | required |
52
+ | cli / low-cost | required | optional | n/a | n/a | n/a | absent |
53
+ | cli / standard | required | optional | n/a | n/a | n/a | absent |
54
+ | cli / full-harness | required | optional | n/a | n/a | n/a | required |
55
+ | mixed / low-cost | required | optional | optional (`skeleton` allowed) | optional | optional | absent |
56
+ | mixed / standard | required | optional | required | optional | optional | absent |
57
+ | mixed / full-harness | required | required | required | required | required | required |
49
58
 
50
59
  Interpretation:
51
60
 
@@ -53,16 +62,16 @@ Interpretation:
53
62
  - `optional`: if present, schema must be valid
54
63
  - `n/a`: absent is normal success
55
64
 
56
- ## non-ui canonical semantics
65
+ ## cli surface canonical semantics
57
66
 
58
- For `surface: non-ui`, the following are normal success when absent:
67
+ For `cli` surface, the following are normal success when absent:
59
68
 
60
69
  - `uiFidelity`
61
70
  - render evidence bundle
62
71
  - browser QA bundle
63
72
  - `runtimeGate.ui`
64
73
 
65
- Contradictory UI-only payloads on `non-ui` are validation errors.
74
+ `ui_bearing: false` specs are not prototyping execution targets. Contradictory UI-only payloads on `cli` surface are validation errors.
66
75
 
67
76
  ## uiFidelity notes
68
77
 
@@ -167,7 +176,7 @@ Rules:
167
176
 
168
177
  ```json
169
178
  {
170
- "surface": "web-ui",
179
+ "surface": "web",
171
180
  "specs": [
172
181
  {
173
182
  "specId": "spec-0001",
@@ -1,6 +1,6 @@
1
1
  # UIX-REV: Comparison Review
2
2
 
3
- Review option comparison and selected anchor screen as independent artifacts.
3
+ Review only `30_option_comparison.md` as the option comparison artifact.
4
4
 
5
5
  ## Comparison Quality (30_option_comparison.md)
6
6
 
@@ -9,22 +9,10 @@ Review option comparison and selected anchor screen as independent artifacts.
9
9
  - Evaluation criteria must reference the 3-layer evaluation family
10
10
  - Rejected/deferred options must have explicit rationale
11
11
  - Reconsideration conditions must be documented for deferred options
12
+ - Do not move selected anchor ownership back into `30_option_comparison.md`
12
13
 
13
14
  ### Trend-derived conversion check
14
15
 
15
16
  - Trend scan results are converted to comparison axes
16
17
  - Stale / overused AI slop avoidance is reflected in comparison criteria
17
-
18
- ## Selected Direction Quality (31_selected_anchor_screen.md)
19
-
20
- - `selected_option` must reference one of the compared options in `30_option_comparison.md`
21
- - `why_selected` must provide rationale aligned with 3-layer evaluation family
22
- - Anchor screen must represent the canonical visual direction
23
- - Downstream strategy (`10_implementation_strategy.md`) must align with selected direction
24
- - Downstream contracts (`40_screen_contracts.md`) must be consistent with selected direction
25
- - Deferred options must document reconsideration conditions
26
-
27
- ### Trend-derived conversion check
28
-
29
- - Trend scan results are reflected in selected direction rationale
30
- - Stale / overused AI slop avoidance is reflected in selection
18
+ - Keep selected-anchor ownership in `31_selected_anchor_screen.md` and scoring-ready field ownership in `50_review_input_bundle.md`
@@ -1,4 +1,4 @@
1
- # UIX-REV: Screen Contracts Review (40_screen_contracts.md)
1
+ # UIX-REV: Screen Contracts Review (`40_screen_contracts.md`)
2
2
 
3
3
  Review screen contracts for canonical 11-field schema completeness.
4
4
 
@@ -22,4 +22,7 @@ Review screen contracts for canonical 11-field schema completeness.
22
22
  - Routes must be unique across all contracts
23
23
  - Required states must include `default`, `loading`, `empty`, and `error`
24
24
  - All 11 fields must be present and non-empty for each screen
25
- - Screen contracts must be consistent with 50_review_input_bundle.md
25
+ - Screen contracts must stay consistent with `31_selected_anchor_screen.md`
26
+ - Screen contracts must stay consistent with Browser QA findings and screen-linked evidence
27
+ - Do not reference legacy filenames such as `40_contracts.md`
28
+ - Keep wording aligned with scoring-ready canonical fields and avoid stale migration vocabulary
@@ -25,9 +25,11 @@ Each evaluation axis must include:
25
25
 
26
26
  ## Evaluation Axes Quality
27
27
 
28
- - Each trend-derived row must include source_translation
29
28
  - Axes must be measurable and verifiable
30
29
  - No generic/boilerplate axes without taste/trend derivation
30
+ - Each axis must define `origin`, `intent`, `why_it_matters`, `score_scale`, `score_anchors.low/mid/high`
31
+ - Each axis must define `positive_signals`, `negative_signals`, `anti_patterns`, `evidence_required`, `minimum_floor`
32
+ - Each axis must define `source_refs`, `goal_refs`, and `review_questions`
31
33
 
32
34
  ## Aggregate Scoring Rules
33
35
 
@@ -46,5 +48,11 @@ Canonical field names (must match `23_design_eval_aggregate.md`):
46
48
  ### Trend-derived conversion check
47
49
 
48
50
  - Trend scan results are converted to scoring axes
49
- - Trend scan results are reflected in selected direction
51
+ - Trend scan results are reflected in selected anchor
50
52
  - Stale / overused AI slop avoidance is reflected in comparison and selection
53
+
54
+ ## Aggregate Review Focus
55
+
56
+ - Review axis quality, overlap, and minimum floors as a single aggregate system
57
+ - Review `source_refs` and `goal_refs` linkage for every scored axis
58
+ - Remove old aggregate vocabulary and keep wording aligned with `23_design_eval_aggregate.md`
@@ -2,22 +2,26 @@
2
2
 
3
3
  Review the UI/UX strategy document (`10_implementation_strategy.md`) for completeness and quality.
4
4
 
5
- This review is scoped to strategy completeness only. Selected direction evaluation belongs to `comparison-review.md`.
5
+ This review is scoped to strategy completeness only. Selected anchor evaluation belongs to `comparison-review.md`.
6
6
 
7
7
  ## Required Fields
8
8
 
9
9
  - `surface`
10
+ - `selection_required`
10
11
  - `decision`
11
- - `why_this_strategy`
12
- - `expected_strengths`
13
- - `known_risks`
14
- - `fit_for_this_product`
15
- - enum validity for `surface` and `decision`
12
+ - `candidate_options`
13
+ - `chosen_option`
14
+ - `rationale`
15
+ - `verification_expectations`
16
+ - `notes_for_reviewer`
16
17
 
17
18
  ## Alignment Check
18
19
 
19
- - Strategy `decision` must be consistent with the selected direction in `31_selected_anchor_screen.md` (cross-reference only; SSOT judgment is in comparison-review)
20
+ - `chosen_option` must exist in `candidate_options`
21
+ - `decision` and `chosen_option` should use coherent vocabulary
22
+ - Strategy `decision` must be consistent with the selected anchor in `31_selected_anchor_screen.md` (cross-reference only; SSOT judgment is in comparison-review)
20
23
  - Strategy must not contradict screen contracts in `40_screen_contracts.md`
24
+ - `selection_required=false` with multiple `candidate_options` should be treated as a warning
21
25
 
22
26
  ## Verdict
23
27