Skip to content

feat(scholar-cli): save browser search results to workspace - #62

Open
dongyikuan919 wants to merge 1 commit into
better-world-ai:mainfrom
dongyikuan919:feat/search-workspace
Open

feat(scholar-cli): save browser search results to workspace#62
dongyikuan919 wants to merge 1 commit into
better-world-ai:mainfrom
dongyikuan919:feat/search-workspace

Conversation

@dongyikuan919

Copy link
Copy Markdown

Summary

  • add --workspace to search-google, search-cnki, and search-wos
  • reuse the existing workspace deduplication and persistence flow
  • document the new flags in the CLI README and paper-research skill

Closes #61

Verification

  • go test ./...
  • go vet ./...
  • go build ./...
  • git diff --check

@xpzouying

Copy link
Copy Markdown
Collaborator

@dongyikuan919 非常抱歉,这个 PR 挂了 10 天没人回,而且经过比"没人回"还糟糕。先把时间线如实摆出来:

时间(08-13) 事件
11:24 我开 issue #61,打上 good first issue
11:47 你提交 commit
11:49 你开了这个 PR
13:12 我自己开了 #63,同样写着 Closes #61
13:37 我自己合并了 #63

good first issue 挂出去 25 分钟你就接了——这是这个标签能有的最好结果。然后我在一个半小时后自己写了一版同样的修复,把 issue 关掉了,全程没有先来这里看一眼有没有人在做,也没在 #61 下面说一声。

这是我的流程问题,不是你的问题。 标了 good first issue 就等于在邀请别人花时间,我不该在没有确认、没有 assign、没有打招呼的情况下自己动手把它做掉。给你造成的白工,抱歉。

你写的东西并没有白费

#63 只改了代码,没动文档。所以到今天为止:

  • scholar-cli/README.md 第 10-12 行,三个命令的用法里仍然没有 --workspace
  • skills/paper-research/SKILL.md 同样没提

也就是说,功能现在是能用的,但**"这个功能存在"这件事,目前只有你这个 PR 写出来了**。这不是安慰奖——这是 main 上实实在在还缺的一块,而且是你先发现要补的。

好消息是收尾很小。我查过了,冲突只在 scholar-cli/cmd/root.go 一个文件:

merge-base = c0061f3
scholar-cli/README.md          → main 自 base 起零改动  ✅ 无冲突
skills/paper-research/SKILL.md → main 自 base 起零改动  ✅ 无冲突
scholar-cli/cmd/root.go        → main 已被 #63 改过     ❌ 唯一冲突点

root.go 的改动丢掉、rebase 到 main,剩下就是 2 files changed, 9 insertions(+), 8 deletions(-),可以直接合。

怎么走,你选

优先权在你——这个 PR 是你的,我不想再一次替你把事做完:

  • A:你自己 rebase(只留两个文档文件),我马上 review 合并。
  • B:你最近没空,回一个 👍 或者一句话,我帮你 rebase 并 force-push 到你的分支,commit author 保留你,然后合并。

哪个都行,不用有压力。这周没回我就默认走 B,不会让这个 PR 再挂下去。

顺手可以一起改的两个小地方(不改也能合):

  1. SKILL.md 里把 --workspace /path/to/workspace 直接拼进了基础示例,会让人以为这个 flag 是必填的。README 用的是 [--workspace DIR] 方括号表示可选,两边统一一下更好。
  2. 占位符 /path/to/workspace 和同一份 SKILL.md 第 135-139 行用的 ~/research/my-survey 不一致,建议统一成后者。

合并后我会把你加进 README 的 all-contributors 表,📖 和 💻 都加。💻 不是送人情:你写的那版实现是完整的、正确的,输出字段还主动和 search-en 对齐了,Verification 一栏 go test / vet / build / git diff --check 四项也都认真跑了。它没能进 main 的原因是我撞了车,不是它不合格。

还有一个留给你

Review 这个 PR 的时候,顺着你改的那块又翻出一个 bug,已经开成 #64 了:search-en 加不加 --workspace 输出的 JSON 结构居然不一样(多包了一层 search),而且和 search-google 那几个命令还不一致。

这个我留着没做,想邀请你来。 理由是它就在你刚碰过的同一个文件、同一块逻辑里,改动量也不大;而且说实话,它是因为有人认真提了 PR、我才去认真读了这段代码才发现的。有兴趣的话在 #64 下面说一声,我 assign 给你(GitHub 要你先在 issue 里留个言才能被 assign)。没兴趣也完全没关系。

再次抱歉,也谢谢你愿意提这个 PR。

@dongyikuan919

Copy link
Copy Markdown
Author

我选 A,已经处理完成:rebase 到最新 main,移除了 scholar-cli/cmd/root.go 的实现改动,只保留 README 和 paper-research SKILL 两个文档文件;同时把 --workspace 标成可选,并将示例路径统一为 ~/research/my-survey。当前提交 ab8fb79,git diff --check 已通过。辛苦 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.

scholar-cli: search-google 缺 --workspace,结果进不了 BibTeX 流程

2 participants