Skip to content

refactor(cli): invite pack 抬头去重(#383)#391

Merged
leeguooooo merged 1 commit into
mainfrom
feat/383-invite-dedup
Jul 12, 2026
Merged

refactor(cli): invite pack 抬头去重(#383)#391
leeguooooo merged 1 commit into
mainfrom
feat/383-invite-dedup

Conversation

@leeguooooo

Copy link
Copy Markdown
Owner

#383 优化 invite.ts:watch / participate 两个 join pack 的抬头(标题+server/channel/scope/charter)完全相同、只模式标签不同,抽成 packHeader(mode) 一处去重。

保守选择:只做输出不变的去重(m3 快照守护,字节一致)。不动 pack 正文——其详尽度是踩坑换来的(false-online/TMPDIR/trellis/任务台账),激进精简拿可靠性冒险不值当。UX 分层若要做另议。

m3 快照无变化(输出字节一致)、cli 全过、tsc 干净。

Refs #383

#383 优化 invite.ts:watch 与 participate 两个 join pack 的抬头(标题栏 + server/
channel/scope/charter)完全相同、只有模式标签不同,却各写一遍。抽成局部 packHeader(mode)
一处,去掉重复。

保守选择:只做输出不变的去重(m3 快照守护,字节一致)。不动 pack 正文——它的详尽度是
踩过的坑换来的(false-online #55/#60、TMPDIR #336、trellis #367、任务台账 #371),
激进精简会拿可靠性冒险,不值当。UX 分层若要做另议(见 #383 我的分析评论)。

验证:m3 快照无变化(证明输出字节一致)、cli 全过、tsc 干净。

Refs #383

Claude-Session: https://claude.ai/code/session_01PgxkZeqJmDge3dYe9W2tPZ
@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

383 - Fully compliant

Compliant requirements:

  • 优化 cli/src/commands/invite.ts
⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
General
修正输出格式

console.log 中添加换行符,以确保输出格式正确。

cli/src/commands/invite.ts [200-203]

     console.log(`${packHeader("观看模式 (watch · readonly)")}
- 
+
  把下面整段发给对方的 agent(Claude Code / Codex)——【观看模式】:只读围观,发送已禁用。
  带 # 的是给它读的说明,不带 # 的是要执行的命令:
Suggestion importance[1-10]: 6

__

Why: 在 console.log 中添加换行符可以改善输出格式,使日志更易读。这是一个小改进,但对可读性有积极影响。

Low

@leeguooooo leeguooooo merged commit 5356333 into main Jul 12, 2026
13 checks passed
@leeguooooo leeguooooo mentioned this pull request Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant