Skip to content

docs(faq): supplement FAQ (zh & en) with community-sourced Q&A#529

Open
FenjuFu wants to merge 3 commits into
iflytek:mainfrom
FenjuFu:faq/add-community-qa
Open

docs(faq): supplement FAQ (zh & en) with community-sourced Q&A#529
FenjuFu wants to merge 3 commits into
iflytek:mainfrom
FenjuFu:faq/add-community-qa

Conversation

@FenjuFu

@FenjuFu FenjuFu commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

FenjuFu added 2 commits June 16, 2026 01:47
Add questions frequently raised in the user community that were not yet
covered in the SkillHub FAQ, for both Chinese and English pages:

- Recommended deployment via the one-line script vs manual image pulls
- Redirected back to login page after deploying (manual deployment)
- Changing the admin password / why env changes need a restart
- Password change/reset requires email code (SMTP setup)
- Skill naming (English only; Chinese names error in OpenClaw)
- Whether unreviewed skills can be downloaded
- Hiding/removing GitHub & GitLab SSO login options
- Built-in Skill Scanner: iFLYTEK integration over Cisco's scanner (Apache-2.0)
- Which cisco-ai-skill-scanner version is used (unpinned in Dockerfile)
- Note that upgrades preserve registered skills; online docs link

Signed-off-by: FenjuFu <fufenjupku@gmail.com>
Signed-off-by: FenjuFu <fufenjupku@gmail.com>
@FenjuFu FenjuFu force-pushed the faq/add-community-qa branch from b6d34da to e3f84b0 Compare June 15, 2026 17:47
Add more community-sourced questions (both Chinese and English):

- Troubleshooting CLI `skillhub publish` returning 400 (name conflict,
  SKILL.md location/frontmatter, namespace membership, etc.)
- Required skill package structure (SKILL.md in root)
- "malformed input" on publish caused by non-UTF-8 / Chinese-path zips
- Per-package file-count limit and how to raise it
- Minimum server version for CLI features (v0.2.7+)
- PostgreSQL-only (no MySQL); plugins not distributable yet
- How to check server/CLI versions and customize via secondary dev

Signed-off-by: FenjuFu <fufenjupku@gmail.com>
@FenjuFu FenjuFu requested review from XiaoSeS and Copilot and removed request for XiaoSeS June 16, 2026 02:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR expands the SkillHub FAQ (Chinese and English) with additional community-sourced Q&A, primarily covering deployment/operations, CLI publishing troubleshooting, packaging constraints, and support links.

Changes:

  • Add multiple new FAQ entries around recommended deployment, login redirect issues, admin password changes, SMTP setup, publishing errors, and skill package constraints.
  • Add clarification text to the upgrade note and add an “Online Docs” support link.
  • Mirror the same FAQ additions in both zh and en variants.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
docs/skillhub/faq.md Adds extensive new Chinese FAQ entries and expands upgrade/support guidance.
docs/skillhub/en/faq.md Adds corresponding English FAQ entries and expands upgrade/support guidance.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/skillhub/faq.md
```

> **注意**:升级前建议先备份数据库和对象存储。数据库迁移由 Flyway 自动执行。
> **注意**:升级前建议先备份数据库和对象存储。数据库迁移由 Flyway 自动执行。升级不会清空数据库,已录入的技能包不会丢失。
Comment thread docs/skillhub/faq.md
SKILLHUB_PUBLISH_MAX_FILE_COUNT=500
```

修改后需重启容器生效。注意 `compose.release.yml` 中也需引用该变量;较旧版本(如 v0.2.6)可能将该值写死,建议升级到最新版本。
Comment thread docs/skillhub/en/faq.md
```

> **Note**: It is recommended to back up the database and object storage before upgrading. Database migrations are handled automatically by Flyway.
> **Note**: It is recommended to back up the database and object storage before upgrading. Database migrations are handled automatically by Flyway. Upgrading does not wipe the database, so already-registered skill packages will not be lost.
Comment thread docs/skillhub/en/faq.md
SKILLHUB_PUBLISH_MAX_FILE_COUNT=500
```

Restart the containers for the change to take effect. Note that `compose.release.yml` must also reference this variable; older versions (e.g. v0.2.6) may hard-code the value, so upgrading to the latest version is recommended.
@dongmucat

Copy link
Copy Markdown
Collaborator

感谢补充这批社区 FAQ,覆盖了部署、CLI 发布、SMTP、Scanner、数据库支持等高频问题,对降低用户排障成本很有帮助。

建议先做几处小修正再合并,主要是避免文档说明和当前实现不一致:

  1. 管理员密码重置说明需要收窄
    docs/skillhub/faq.md / docs/skillhub/en/faq.md

    当前文档写到管理员可通过 .env.release 重置密码,但 bootstrap admin 逻辑只会在 credential 不存在时写入密码。实例已初始化后,仅修改 .env.release 并重启不会更新已有密码。建议说明该方式仅适用于首次初始化,或明确需要清理已有 credential 后才会重新初始化。

  2. 文件数上限环境变量说明需要和 release compose 对齐
    docs/skillhub/faq.md / docs/skillhub/en/faq.md

    FAQ 建议通过 SKILLHUB_PUBLISH_MAX_FILE_COUNT=500 覆盖文件数上限,但当前 compose.release.yml 的 server environment 没有传入该变量。用户只改 .env.release 时不会生效。建议同步更新 compose.release.yml,或在 FAQ 中明确需要把该变量加入 server 容器环境变量。

  3. 未审核 skill 下载权限描述建议更精确
    docs/skillhub/faq.md / docs/skillhub/en/faq.md

    “只要拥有可查看权限,一般都可以下载”目前有些宽。当前下载服务对版本状态仍有限制,例如只允许 PUBLISHEDUPLOADEDPENDING_REVIEW 等可下载状态。建议改成按版本状态和权限共同判断的描述,避免用户把“可查看”等同于“必然可下载”。

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.

3 participants