Skip to content

[codex] improve xhs and non-video workflow usability#50

Draft
haoabcde wants to merge 1 commit into
XBuilderLAB:mainfrom
haoabcde:codex/official-usability-sync
Draft

[codex] improve xhs and non-video workflow usability#50
haoabcde wants to merge 1 commit into
XBuilderLAB:mainfrom
haoabcde:codex/official-usability-sync

Conversation

@haoabcde

Copy link
Copy Markdown

Summary

  • add XHS starter rubrics and a post template for users who pick short-text + xhs
  • add a maintainer verification script for install lists, shell/python syntax, diff_pct, and XHS writing rules
  • make non-video flows easier to use by preventing cheat-shoot from being used for article/XHS publishing
  • expand .gitignore for local agent state, runtime content artifacts, and non-video project outputs

Scope guard

  • no user content/output files are included
  • no khazix language-style skill or private style examples are included
  • this PR keeps the upstream flat repository structure

Validation

  • bash tools/verify-package.sh
  • git diff --cached --check
  • staged leak checks for content directories and khazix style references

@Jooonnn

Jooonnn commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

注意到这个 PR 还是 draft,等你标 ready 再合。先 review 了一下,两个建议(非阻塞):

  1. cold-start 小红书用户走不到新 startercheat-init/SKILL.md 的 starter 选择规则里,路由到 xhs-post-zero.md 要求 content_form=short-textenabled_perf_adaptersxhs-explore。但 enabled_perf_adapters 只对"发过内容"(Q2=b)的用户填充;真正的 cold-start 小红书用户答 Q2=a,enabled_perf_adapters=[],于是落到 else 分支拿到 opinion-video-zero.md 占位——也就是新写的 cold-start 小红书 rubric 恰恰对它的目标用户不可达。cheat-seedskills/cheat-seed/SKILL.md)是用"用户明确要小红书"做了出口,建议 cheat-init 也加同样的逃生通道。
  2. verify 脚本漏覆盖tools/verify-package.sh 的 shell 语法检查 find 只覆盖 adapters/perf-data/*/run.shadapters/script-extraction/whisper/run.sh 被静默排除——那里的语法错误会漏检。建议放宽 glob。

其余(scope guard 无内容泄漏、set -uo pipefail 不带 -e 的设计、starter rubric 结构)都没问题。

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