[Bug]: Removing Vibe event hooks deletes user comments that follow a Spec Kit hook block

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#5,079 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
cli

Hướng nghiên cứu

Đọc src/specify_cli/integrations/vibe/__init__.py, đặc biệt là các dòng 440–490 về phát hiện khối được quản lý và _without_managed_hooks, sau đó xem lại remove_vibe_event_hooks cùng các bài kiểm thử liên quan trong tests/integrations/test_integration_vibe.py. Chạy các bài kiểm thử tích hợp Vibe bao quát việc giữ lại chú thích và xóa; hoàn tất khi chú thích và hook của người dùng vẫn còn sau khi xóa, và một tệp chứa nội dung của người dùng vẫn được giữ lại.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug-assess invalid triage-must-have
Bug Description

VibeIntegration finds the [[hooks]] blocks that Spec Kit owns (marked speckit_marker = true) and treats each one as running until the next table header or the end of the file (src/specify_cli/integrations/vibe/__init__.py:440-470). Comment and blank lines after a managed block therefore count as part of it. When the hooks are removed, those user comments are deleted. If a comment is the only user content left, remove_vibe_event_hooks deletes the whole .vibe/hooks.toml file. _merge_managed_hooks, used when hooks are written again, uses the same block detection and drops the comments as well.

Steps to Reproduce

From the root of a fresh clone, after uv sync, save the script below as repro.py and run it with uv run python repro.py.

import tempfile
from pathlib import Path
from specify_cli.integrations import get_integration

vibe = get_integration("vibe")
managed = (
    '[[hooks]]\nname = "speckit-pre_tool-validate"\ntype = "pre_tool"\n'
    'command = "c"\ntimeout = 65\nspeckit_marker = true\n'
)
cases = {
    "comment before a user hook": managed
    + '\n# NOTE: keep guard-bash first\n[[hooks]]\nname = "user-hook"\n'
    'type = "pre_tool"\ncommand = "guard-bash"\n',
    "trailing user comment only": managed + "\n# my notes about vibe hooks\n",
}
for label, text in cases.items():
    root = Path(tempfile.mkdtemp())
    path = root / ".vibe" / "hooks.toml"
    path.parent.mkdir()
    path.write_text(text)
    vibe.remove_vibe_event_hooks(root, ensure_safe_destination=lambda p: None)
    print(f"[{label}]")
    print("  file exists after removal:", path.exists())
    print("  content:", repr(path.read_text()) if path.exists() else None)

The script writes a .vibe/hooks.toml containing a Spec Kit managed hook followed by user content, then calls remove_vibe_event_hooks, the method used when Vibe event hooks are removed.

Expected Behavior

Only the Spec Kit managed hook is removed. The user's comment and the user's hook stay, and a file that still holds a user comment is kept. The _without_managed_hooks docstring says it preserves "every other source slice verbatim" (src/specify_cli/integrations/vibe/__init__.py:477-490), and the remove_vibe_event_hooks docstring says it removes "only Specify-owned Vibe hooks and delete[s] an owned-only file" (vibe/__init__.py:553-590). The tests test_empty_events_preserve_comments_only_file and test_removal_preserves_user_comment_with_marker_text (tests/integrations/test_integration_vibe.py:496-528) treat user comments as content to keep.

Actual Behavior
[comment before a user hook]
  file exists after removal: True
  content: '[[hooks]]\nname = "user-hook"\ntype = "pre_tool"\ncommand = "guard-bash"\n'
[trailing user comment only]
  file exists after removal: False
  content: None

In the first case the line # NOTE: keep guard-bash first is gone. In the second case the file is deleted together with the user's comment.

Specify CLI Version

1.2.4.dev0 (main at 27d14e2)

AI Agent

Mistral Vibe

Operating System

Linux 6.18 (Ubuntu container)

Python Version

Python 3.12.3

Additional Context

A comment placed after a managed block is likely to describe the user content that follows it, as in the first case, so attaching it to the managed block loses user content.

Related: #4646 (merged) added this removal path. A review comment on that pull request flagged the comment loss, and no later commit addressed it.

Also reproduced on macOS (Darwin 25.6, arm64) with Python 3.14.4 and uv run python -I repro.py against 27d14e2; the output matches the above.

AI Disclosure

Found and reproduced with Claude Code (model: Claude Opus 4.8), run autonomously under human supervision. The human reporter re-ran every reproduction above before filing.

Ngôn ngữ chính
Python
Star
149k
Fork
13.4k
Merge trung bình
1 ngày 5 giờ
Pull request đã merge (30 ngày)
223

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của github/spec-kit

Tất cả issue của github/spec-kit

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.