Skip to content

fix: remove restore-project prompt from taptap-lookup skill output - #154

Merged
lishuceo merged 1 commit into
mainfrom
feat/taptap-lookup-skill
Mar 17, 2026
Merged

fix: remove restore-project prompt from taptap-lookup skill output#154
lishuceo merged 1 commit into
mainfrom
feat/taptap-lookup-skill

Conversation

@lishuceo

Copy link
Copy Markdown
Owner

Summary

  • 去掉 taptap-lookup skill 输出中的 /restore-project 后续操作提示,防止滥用

PR #153 merge 时遗漏的 fix commit。

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@lishuceo
lishuceo force-pushed the feat/taptap-lookup-skill branch from ddfa70c to 00ecf11 Compare March 17, 2026 04:36
@claude

claude Bot commented Mar 17, 2026

Copy link
Copy Markdown

Claude finished @lishuceo's task in 35s —— View job


Review: PR #154

  • Check for previous review comments — none found
  • Review the diff
  • Post review summary

Approved — Clean, minimal change that does exactly what it says.

The PR replaces the "后续操作提示" section (which suggested using /restore-project) with a "注意事项" section that explicitly instructs the skill to only output the query result table. This correctly prevents the skill from surfacing the /restore-project command to users.

No bugs, no security issues, no architectural concerns. LGTM.

@lishuceo
lishuceo merged commit 4773983 into main Mar 17, 2026
4 checks passed
@lishuceo
lishuceo deleted the feat/taptap-lookup-skill branch March 17, 2026 04:38
lishuceo added a commit that referenced this pull request Apr 8, 2026
)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant