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.
- package/README.md +7 -5
- package/assets/init/.qfai/assistant/agents/frontend-engineer.md +2 -2
- package/assets/init/.qfai/assistant/agents/product-experience-architect.md +2 -2
- package/assets/init/.qfai/assistant/agents/product-surface-reviewer.md +1 -1
- package/assets/init/.qfai/assistant/skills/qfai-discussion/SKILL.md +44 -18
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/01_Context.md +9 -0
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/03_Story-Workshop.md +1 -1
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/04_Sources.md +6 -6
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/14_Review-Request.md +7 -7
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/review/Rxx_reviewer.md +2 -2
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/review/review_request.md +2 -2
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/10_implementation_strategy.md +31 -13
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/20_trend_scan.md +41 -0
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/23_design_eval_aggregate.md +12 -0
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/40_screen_contracts.md +1 -1
- package/assets/init/.qfai/assistant/skills/qfai-discussion/templates/uiux/50_review_input_bundle.md +2 -0
- package/assets/init/.qfai/assistant/skills/qfai-implement/SKILL.md +6 -2
- package/assets/init/.qfai/assistant/skills/qfai-prototyping/SKILL.md +264 -34
- package/assets/init/.qfai/assistant/steering/agent-catalog.yml +1 -1
- package/assets/init/.qfai/assistant/steering/agent-routing.yml +1 -1
- package/assets/init/.qfai/assistant/steering/manifest.md +4 -7
- package/assets/init/.qfai/assistant/steering/product.md +6 -6
- package/assets/init/.qfai/assistant/steering/review-profiles.yml +3 -0
- package/assets/init/.qfai/assistant/steering/ui-definition-protocol.md +2 -2
- package/assets/init/.qfai/contracts/ui/README.md +2 -2
- package/assets/init/.qfai/discussion/README.md +14 -22
- package/assets/init/.qfai/evidence/README.md +21 -12
- package/assets/uix-rev/comparison-review.md +3 -15
- package/assets/uix-rev/contracts-review.md +5 -2
- package/assets/uix-rev/scoring-review.md +10 -2
- package/assets/uix-rev/strategy-review.md +11 -7
- package/dist/cli/index.cjs +1993 -1279
- package/dist/cli/index.cjs.map +1 -1
- package/dist/cli/index.mjs +1930 -1216
- package/dist/cli/index.mjs.map +1 -1
- package/dist/index.cjs +1989 -1269
- package/dist/index.cjs.map +1 -1
- package/dist/index.d.cts +75 -62
- package/dist/index.d.ts +75 -62
- package/dist/index.mjs +1926 -1207
- package/dist/index.mjs.map +1 -1
- package/package.json +1 -1
- 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
|
|
42
|
-
|
|
|
43
|
-
|
|
|
44
|
-
|
|
|
45
|
-
|
|
|
46
|
-
|
|
|
47
|
-
|
|
|
48
|
-
|
|
|
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
|
-
##
|
|
65
|
+
## cli surface canonical semantics
|
|
57
66
|
|
|
58
|
-
For `surface
|
|
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 `
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
-
- `
|
|
12
|
-
- `
|
|
13
|
-
- `
|
|
14
|
-
- `
|
|
15
|
-
-
|
|
12
|
+
- `candidate_options`
|
|
13
|
+
- `chosen_option`
|
|
14
|
+
- `rationale`
|
|
15
|
+
- `verification_expectations`
|
|
16
|
+
- `notes_for_reviewer`
|
|
16
17
|
|
|
17
18
|
## Alignment Check
|
|
18
19
|
|
|
19
|
-
-
|
|
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
|
|