Repository navigation
feat: 文件整理 carl-file-organizer v0.3 并入 carl-skills(一页报告、三色两层、整机盘点、Windows 兼容) - #7
Merged
Merged
Conversation
…三色报告模板与 build_report Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
… decisions Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
CI 三平台跑起来之后,macOS 之外的两条腿红了,逐条查下来只有一条是测试自己
的问题,其余都是代码里真的假设了 macOS:
- 盘点分类看的是整条绝对路径,home 之上的目录名也算数。CI 的假 home 造在
/tmp 底下,一个 tmp 段就把整台机器判成缓存,三色一起塌成 green。分类改成
只看 home 以下的那一段(scoped),Windows 的 %TEMP% 在 AppData 里,同一个
坑那边更容易踩。
- 计划里的绝对路径是会跨机器旅行的:Mac 上写的计划在 Windows 上读,字符串
里是正斜杠,还原出来的 home 是反斜杠,隐私改写一条都没命中,报告页就会
带着原机器的路径发出去。改写现在两种分隔符拼写都认。
- path_forms 只生成本机拼写的绝对路径。Windows 上配置文件里两种写法都有
(Git Bash 写正斜杠,PowerShell 写反斜杠),漏一种就是漏掉一处引用。
- refuse_root 不认盘符根:REFUSED_ABSOLUTE 里只有 "/",C:\ 得靠 anchor 判。
- 中文输出撞上 Windows 重定向后的老代码页会抛 UnicodeEncodeError,被 CLI
边界吞成退出码 2——计划明明已经写好了,报的却是失败。改成转义不可表示的
字符,落盘的 JSON 一直是 UTF-8,不受影响。
测试侧只动平台专属能力和写死了 POSIX 口径的断言:执行位、chmod 000、
AppleScript 转义要用的引号文件名在 Windows 上做不到,按理由 skip;
os.geteuid、Path("/opt/...").is_absolute()、临时目录一定在 home 之外这三个
假设改成跨平台写法。没有为了变绿删掉任何一条断言。
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
把文件整理从独立仓库并进 carl-skills,作为 collection-native skill。
scan → report → apply单入口,旧格式批准文件仍认🤖 Generated with Claude Code