Visitar URL original
feat(arch): org: 前缀 + 组图标兜底(公司/机构图标在私有仓,公开层画自绘的通用剪影) by edwinjhlee · Pull Request #19 · x-cmd/draw · GitHub
Skip to content

feat(arch): org: 前缀 + 组图标兜底(公司/机构图标在私有仓,公开层画自绘的通用剪影) - #19

Merged
edwinjhlee merged 2 commits into
mainfrom
feat/org-icon-group
Nov 12, 2026
Merged

edwinjhlee merged 2 commits into
mainfrom
feat/org-icon-group

Conversation

@edwinjhlee

Copy link
Copy Markdown
Member

公司/机构图标(org: 前缀)那族素材住在私有仓的内部图库里 —— 商标图形不能公开分发。公开请求查不到 → 画组图标(自绘的通用机构剪影),不是空框。

改了什么

  • ICON_FIELD_RE 放行 org:(src/render/archmap.ts)。
  • 新增 src/render/orgicon.ts:组图标 = 自绘的 24×24 平顶办公楼剪影(墙体 + 四扇窗 + 门廊缺口)。平顶才像「机构/公司」—— 坡屋顶会读成「房子」,在域名后缀的图里恰恰是最不该出现的联想。颜色跟节点的语义色走(var(--d-sN)),明暗两套皮肤都合适。
  • 兜底:
    • 大图标版(/draw/cloud)→ 画组图标,节点仍是图标节点,不是普通方框;
    • arch2 的角标位 → 画组图标(那格说的是「这是个机构」,比类型字母有信息量);
    • 不认识的 aws: 图标(拼错)行为完全不变。
  • 新增注入点:ArchOpts.icons / ArchUrlOpts.icons(名字 → data URI,优先于内置图标库)—— 私有仓的内部图库走这条路把真 logo 塞进来。图标解析一次,渲染前填进节点的 iconData。公开仓自己不带任何第三方素材。
  • 测试:6 条新用例。全量 403 passed,npm run check 干净。

不在这次的范围

tier / 内部图库 / 真 logo 都没做 —— 设计见私有仓的决策记录。私有仓那边已经有一条案例用上了这个前缀(/draw/demo 里的「域名后缀的运营方」,数据取自 IANA 根区库)。

🤖 Generated with Claude Code

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@edwinjhlee
edwinjhlee merged commit 34e734d into main Nov 12, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants