wsb1224
e3479f0546
# 问题 修复 文件 1 前端构建失败（引号错误） size="small type=" → size="small" type=" PosterHistoryPage.vue 2 migrate_014 ORM vs 缺失列 全部改为原始 SQL，不再引用 ORM 模型 migrate_014.py 3 cleanup 字段名错误 output_path → ppt_path cleanup.py 4 文案生成 case 越权 添加 case.user_id != user_id 校验 poster/service.py 5 存储路径未接通持久化卷 全部改用 get_storage_root()（默认 /app/api/storage/insurance） config.py, ppt/routes.py, poster/service.py, poster/tasks.py 高风险问题修复 # 问题 修复 文件 6 migrate_019 rollback 撤销成功字段 每个 ALTER 后立即 commit，失败只回滚当前语句 migrate_019.py 7 迁移锁 Windows 不兼容 + 句柄未持久化 全局变量保存锁句柄，支持 Windows msvcrt api/insurance/db/__init__.py 8 PDF 校验异常时放行 异常返回 False（文件损坏） security.py 9 健康检查始终返回成功 缺少关键资源时返回 503 + missing 列表 poster/routes.py 10 短密钥掩码泄露原值 ≤4 字符返回 **** ppt_admin_service.py 11 设置无键名白名单 添加 _ALLOWED_SETTING_KEYS 白名单 ppt_admin_service.py 12 容器重启任务永久 stuck 添加 recover_stale_tasks() 启动恢复函数 poster/tasks.py, ppt/parse_worker.py
98 lines
5.1 KiB
Markdown
98 lines
5.1 KiB
Markdown
> **Additional context needed**: quality bar and shipping constraints.
|
|
|
|
Polish is refinement, never concealed redesign. Preserve the incumbent visual world, content, behavior, and everything outside scope. If the concept itself is wrong, say so and recommend redesign or `bolder` instead of smuggling in a replacement.
|
|
|
|
A detector result is defect evidence, not proof of quality. Inspect the rendered experience and real interaction path.
|
|
|
|
## 1. Establish the system
|
|
|
|
Read DESIGN.md and representative tokens, shared components, patterns, and neighboring flows. If no formal system exists, use coherent project conventions.
|
|
|
|
Classify each drift before fixing it:
|
|
|
|
- **missing token:** the system needs a reusable value;
|
|
- **one-off implementation:** an existing shared component or pattern should replace it;
|
|
- **conceptual mismatch:** the flow, information architecture, or hierarchy differs from comparable product areas;
|
|
- **local defect:** the implementation is simply incomplete or inconsistent.
|
|
|
|
Fix the cause at the narrowest correct level. Ask when a binding system principle cannot be inferred.
|
|
|
|
## 2. Gather the evidence
|
|
|
|
Use the feature yourself at representative desktop and mobile sizes. Determine:
|
|
|
|
- whether the path is functionally complete;
|
|
- the intended quality bar and time available;
|
|
- known constraints or deliberately unfinished work;
|
|
- the states, content lengths, roles, and input methods users will actually encounter.
|
|
|
|
If a prior critique exists, use it as one input:
|
|
|
|
```bash
|
|
node .claude/skills/impeccable/scripts/critique-storage.mjs latest "<resolved target>"
|
|
```
|
|
|
|
Exit 0 returns the latest snapshot; incorporate relevant P0/P1 findings and name the snapshot read. Exit 2 means none exists. Perform an independent pass either way.
|
|
|
|
## 3. Triage
|
|
|
|
Separate functional defects from cosmetic ones and fix in this order:
|
|
|
|
1. broken or blocked tasks, data loss, misleading state, and inaccessible paths;
|
|
2. missing loading, empty, error, success, disabled, and permission states;
|
|
3. flow, hierarchy, responsive, and design-system drift;
|
|
4. visual and motion inconsistencies;
|
|
5. code and asset cleanup.
|
|
|
|
Do not perfect one corner while leaving the rest below the same quality bar.
|
|
|
|
## 4. Polish the whole path
|
|
|
|
### Flow and hierarchy
|
|
|
|
- Match neighboring mental models, terminology, disclosure, routing, save behavior, and optimistic or pessimistic patterns.
|
|
- Make the primary task and current state obvious without flattening every element to equal weight.
|
|
- Ensure arrival, transition, empty, and recovery paths connect instead of behaving as isolated screens.
|
|
|
|
### Layout and type
|
|
|
|
- Align to the project's grid and spacing scale; fix optical as well as mathematical alignment.
|
|
- Group related content tightly and separate distinct groups generously.
|
|
- Keep same-role typography consistent; test measure, wrapping, localization expansion, zoom, and font loading.
|
|
- Verify every supported viewport rather than correcting only the current screenshot.
|
|
|
|
### Color, imagery, and icons
|
|
|
|
- Use semantic tokens and stable color meanings across themes.
|
|
- Verify text, control, and focus contrast in every state.
|
|
- Keep icon families, stroke/weight, sizing, and optical alignment coherent.
|
|
- Prevent image layout shift; use correct aspect ratios, responsive sources, and useful alt text.
|
|
|
|
### Interaction and state
|
|
|
|
- Every control needs appropriate default, hover, focus, active, disabled, loading, error, and success behavior.
|
|
- Preserve visible keyboard focus, logical tab order, labels, and platform-appropriate touch targets.
|
|
- Keep motion coherent, interruptible, and performant. Do not add animation merely to make polish visible.
|
|
- Validate long, missing, localized, offline, slow, and permission-limited content where the product can encounter it.
|
|
|
|
### Content and code
|
|
|
|
- Keep terminology, capitalization, punctuation, and factual copy consistent. Ask before changing claims.
|
|
- Remove debug output, dead code, unused imports, obsolete styles, and polish-created duplication.
|
|
- Replace custom implementations with shared components where the system owns the pattern.
|
|
- Promote genuinely reusable values to tokens; do not create a system abstraction for one local exception.
|
|
|
|
## 5. Verify and finish
|
|
|
|
Walk the complete path again with mouse, keyboard, and touch where applicable. Check:
|
|
|
|
- mobile, intermediate, and wide layouts;
|
|
- loading, empty, error, success, disabled, long-content, and missing-content states;
|
|
- zoom, contrast, focus, semantics, and screen-reader names;
|
|
- console errors, layout shift, interaction latency, image loading, and supported browsers;
|
|
- agreement with DESIGN.md, neighboring features, and the user's scope.
|
|
|
|
Follow the quality guidance supplied by `context.mjs` and hooks, then run any other relevant QA commands. Context requests a manual scan only when no automatic detector is active; never add another detector pass. Fix real defects and document only narrow intentional exceptions. A clean scan does not replace visual judgment.
|
|
|
|
Finish with a source diff: remove accidental churn, orphaned code, redundant values, and temporary artifacts. Ship only when the feature is functionally complete and consistently finished across the path.
|