@lark-apaas/coding-steering 0.1.18-dev.857e860 → 0.1.18-dev.9dbf919

package/steering/design-html/skills/slide-deck/SKILL.md

@@ -19,10 +19,9 @@ metadata:
 
 ## PPTX/POTX 模板附件前置步骤
 
-如果用户上传了 `.pptx` 或 `.potx`，并要求制作/生成/改做一个 PPT、演示文稿、slides、deck，先判断附件是否是模板或视觉参考。
+如果对话里出现 `.pptx` 或 `.potx` 附件，并且当前任务是制作/生成/改做一个 PPT、演示文稿、slides、deck，**默认附件就是模板或视觉参考，先运行 `pptx-style-extract` skill**，读取产出的 `design.md`、`layouts.md` 和 `assets/`，再开始写 deck。不需要等用户再说“按这个模板”。
 
-- 命中“按这个模板 / 照附件风格 / 参考这个 PPT / 保持同款视觉 / 基于这个模板”等意图时，先调用 `pptx-style-extract` skill，读取产出的 `design.md`、`layouts.md` 和 `assets/`，再开始写 deck。
-- 只总结、翻译、提取内容、审阅已有 PPTX 时，不需要抽取风格。
+- 只有用户明确要求只总结、只翻译、只提取内容、只审阅或直接修改已有 PPTX 时，才跳过风格抽取。
 - 不要用 `SummarizeAttachmentOrFile` 的 Markdown/文本摘要替代风格抽取；文本摘要不包含母版、色板、字体、版式坐标和素材角色。
 
 ## 动手前先问
@@ -51,6 +50,7 @@ deck-stage 组件会对每个 slotted 子元素做绝对定位——**绝不**
 
 - 先读 `design.md` 的 Usage / Hard Rules / colors / typography / components / assets / safe-area。
 - 再读 `layouts.md`，用其中的 layout archetype 和 slots 坐标生成页面；坐标单位已经是 px@1920，不需要 EMU 或 pt 换算。
+- `layouts.md` 的 slot 中，`box` 只负责几何，机械展开为 `left/top/width/height`；`css` 是模板已转译好的最终渲染样式，原样写入元素 style。它优先于本 skill 的通用字号/间距建议，不得重新解释成另一套 token，也不得丢掉其中的 `padding`、对齐、行高、字距或旋转。
 - 消费并落实 colors / typography / spacing / rounded / safe-area 等样式 token；可以用 CSS variables、类名或内联样式承载，但最终页面必须看得出这些 token 被系统性使用，而不是另起一套视觉系统。
 - 背景和版式必须成对消费：选择某个 layout archetype 时，同时采用它声明的 `background`、`text_safe`、`avoid` / `pairing_rule`。标题、正文、图表、表格、卡片、时间线及其容器外接矩形不得进入背景禁放区；如果内容与背景主体冲突，换页型、拆页或缩小内容区域，不要只换背景色或把容器铺到禁放区。
 - 把包内 `assets/` 复制到项目内相对目录并引用复制后的路径；最终 HTML 禁止引用 `/tmp` 或本机绝对路径。
@@ -144,6 +144,21 @@ deck-stage 组件会对每个 slotted 子元素做绝对定位——**绝不**
 5. **把这套 token 当成每页的内容预算**：在上述数值下，一页正文区大约容纳 14 行正文、或 6 个两行 bullet——在 scratchpad 排内容时就按预算裁剪，而不是写完再看塞不塞得下。装不下的处置顺序是**拆页 > 删内容 > 换更省空间的版式**；缩小字号是最后手段，且绝不越过 24px 下限——靠缩字塞进去的页，只是把溢出换成了后排看不清。反过来，内容远少于预算的页按「视觉平衡」的出路增密或合并，而不是放大字号去撑面积。
 6. 构建幻灯片，牢记每张幻灯片既是设计练习也是文案练习。在版式、文字内容和语调方面给予每张幻灯片应有的关注。遵循上述原则，确保每张幻灯片能独立成立；一个只看这一页的人，应当无需其他上下文就能理解其高层含义。
 
+## 提交前资源完整性门禁
+
+最终一次写入 HTML/CSS 后、调用 `run_commit` 前，必须从项目根目录运行：
+
+```bash
+python3 <本skill目录>/scripts/check_local_references.py index.html
+```
+
+把 `<本skill目录>` 换成本 skill 的实际所在目录。脚本从最终 `index.html` 出发，递归检查 HTML/CSS 引用的每个项目内脚本、样式、图片、字体等文件是否真实存在；外部 URL 不做离线探测。
+
+- 只有输出 `RESOURCE_CHECK: PASS` 才能提交。
+- 输出 `RESOURCE_CHECK: FAIL` 时，先重新复制或修正列出的项目内文件，再原样重跑；不得删除引用来掩盖仍在使用的资源。
+- 该检查针对最终工作区状态。较早执行过复制命令或检查，不能证明最终提交完整。
+- `run_commit` 的静态检查跳过参数不能替代本门禁，也不能用来绕过失败；提交时必须包含检查通过所依赖的全部项目内文件。
+
 ## 验证要点
 
 审阅时，用幻灯片构图规则——而非网页布局直觉——来检查版面。底部留白是不是缺陷，用「留白 ≠ 空洞」的归属判据：内容自身完整、下方是无边框的整块呼吸空间，这是正确的幻灯片构图——不要出于网页直觉把 `flex-start` 改成 `center`；空白被元素边界圈占的，是被动空洞，按「视觉平衡」的出路修。

package/steering/design-html/skills/slide-deck/scripts/check_local_references.py

@@ -0,0 +1,179 @@
+#!/usr/bin/env python3
+
+import re
+import sys
+from html.parser import HTMLParser
+from pathlib import Path
+from urllib.parse import unquote, urlsplit
+
+
+CSS_URL_PATTERN = re.compile(r"""url\(\s*(['"]?)(.*?)\1\s*\)""", re.IGNORECASE)
+CSS_IMPORT_PATTERN = re.compile(
+    r"""@import\s+(?:url\(\s*)?(['"])(.*?)\1\s*\)?""",
+    re.IGNORECASE,
+)
+CSS_COMMENT_PATTERN = re.compile(r"/\*.*?\*/", re.DOTALL)
+HREF_RESOURCE_TAGS = {"image", "link", "use"}
+SRC_RESOURCE_TAGS = {
+    "audio",
+    "embed",
+    "iframe",
+    "img",
+    "input",
+    "script",
+    "source",
+    "track",
+    "video",
+}
+
+
+class ReferenceParser(HTMLParser):
+    def __init__(self) -> None:
+        super().__init__()
+        self.references: set[str] = set()
+        self.in_style = False
+
+    def handle_starttag(self, tag: str, attrs: list[tuple[str, str | None]]) -> None:
+        tag = tag.lower()
+        if tag == "style":
+            self.in_style = True
+        attributes = dict(attrs)
+        if tag in HREF_RESOURCE_TAGS and attributes.get("href"):
+            self.references.add(attributes["href"])
+        if tag in SRC_RESOURCE_TAGS and attributes.get("src"):
+            self.references.add(attributes["src"])
+        if tag == "object" and attributes.get("data"):
+            self.references.add(attributes["data"])
+        if tag == "video" and attributes.get("poster"):
+            self.references.add(attributes["poster"])
+
+        srcset = attributes.get("srcset")
+        if tag in {"img", "source"} and srcset:
+            self.references.update(
+                candidate.strip().split()[0]
+                for candidate in srcset.split(",")
+                if candidate.strip()
+            )
+
+        style = attributes.get("style")
+        if style:
+            self.references.update(css_references(style))
+
+    def handle_endtag(self, tag: str) -> None:
+        if tag.lower() == "style":
+            self.in_style = False
+
+    def handle_data(self, data: str) -> None:
+        if self.in_style:
+            self.references.update(css_references(data))
+
+
+def css_references(content: str) -> set[str]:
+    content_without_comments = CSS_COMMENT_PATTERN.sub("", content)
+    references = {
+        match.group(2).strip() for match in CSS_URL_PATTERN.finditer(content_without_comments)
+    }
+    references.update(
+        match.group(2).strip() for match in CSS_IMPORT_PATTERN.finditer(content_without_comments)
+    )
+    return references
+
+
+def resolve_local_reference(
+    raw_reference: str,
+    source: Path,
+    project_root: Path,
+) -> Path | None:
+    reference = raw_reference.strip()
+    parsed = urlsplit(reference)
+    if (
+        not reference
+        or reference.startswith(("#", "//"))
+        or (parsed.scheme and parsed.scheme.lower() != "file")
+        or parsed.netloc
+    ):
+        return None
+
+    path_text = unquote(parsed.path)
+    if not path_text:
+        return None
+
+    if parsed.scheme.lower() == "file":
+        return Path(path_text).resolve()
+    if path_text.startswith("/"):
+        return (project_root / path_text.lstrip("/")).resolve()
+    return (source.parent / path_text).resolve()
+
+
+def collect_references(entry: Path, project_root: Path) -> list[tuple[Path, str, Path]]:
+    pending = [entry]
+    visited: set[Path] = set()
+    local_references: list[tuple[Path, str, Path]] = []
+
+    while pending:
+        source = pending.pop()
+        if source in visited or not source.is_file():
+            continue
+        visited.add(source)
+
+        content = source.read_text(encoding="utf-8")
+        if source.suffix.lower() == ".css":
+            references = css_references(content)
+        else:
+            parser = ReferenceParser()
+            parser.feed(content)
+            references = parser.references
+
+        for raw_reference in sorted(references):
+            target = resolve_local_reference(raw_reference, source, project_root)
+            if target is None:
+                continue
+            local_references.append((source, raw_reference, target))
+            if target.suffix.lower() == ".css" and target.is_file():
+                pending.append(target)
+
+    return local_references
+
+
+def display_path(path: Path, project_root: Path) -> str:
+    try:
+        return path.relative_to(project_root).as_posix()
+    except ValueError:
+        return str(path)
+
+
+def main() -> int:
+    if len(sys.argv) != 2:
+        print("usage: check_local_references.py <entry.html>", file=sys.stderr)
+        return 2
+
+    project_root = Path.cwd().resolve()
+    entry = (project_root / sys.argv[1]).resolve()
+    if not entry.is_file():
+        print(f"RESOURCE_CHECK: FAIL entryNotFound={sys.argv[1]}")
+        return 1
+
+    missing: list[tuple[Path, str, Path]] = []
+    for source, raw_reference, target in collect_references(entry, project_root):
+        try:
+            target.relative_to(project_root)
+        except ValueError:
+            missing.append((source, raw_reference, target))
+            continue
+        if not target.is_file():
+            missing.append((source, raw_reference, target))
+
+    if not missing:
+        print("RESOURCE_CHECK: PASS")
+        return 0
+
+    print(f"RESOURCE_CHECK: FAIL missingLocalReferences={len(missing)}")
+    for source, raw_reference, target in missing:
+        source_name = display_path(source, project_root)
+        target_name = display_path(target, project_root)
+        print(f"- {source_name}: {raw_reference} -> {target_name}")
+    return 1
+
+
+if __name__ == "__main__":
+    raise SystemExit(main())