refactor(kitty-logging): move watcher to scripts/ and symlink it #71
No Reviewers
Labels
Clear labels
Area/CI
Area/Completions
Area/Components
Area/Config
Area/Docs
Area/Functions
Area/Integrations
Area/Prompt & Theme
Area/Scripts
Area/Tests
Compat/Breaking
Kind/Bug
Kind/Chore
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Performance
Kind/Refactor
Kind/Security
Kind/Testing
good first issue
help wanted
.github/workflows and repository automation
completions/ - tab-completion scripts
The opinionated-component system (C1-C6)
config.fish and conf.d/ - startup and environment
docs/manual/ and the generated manual, man page, and site
functions/ - user-facing functions and helpers
integrations/ - third-party tool wiring
themes/ and prompt appearance
scripts/ - standalone helper scripts
tests/ - the syntax lint and functional suite
Breaking change that won't be backward compatible
Something is not working
Tooling, dependencies, and housekeeping
Documentation changes
Improve existing functionality
New functionality
Makes existing behavior faster or lighter
Restructures code without changing behavior
This is security issue
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
Well-scoped and self-contained; a good place to start
Maintainer is looking for someone to pick this up
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
rootiest (Rootiest)
Clear assignees
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: rootiest/fish-config#71
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
Why: The Kitty watcher lived alone in a single-purpose
kitty/directory and was copied into the Kitty config dir, which required a whole version-staleness apparatus (version markers,-gtcomparison, "updated" vs "installed" messaging) just to keep the copy fresh. A symlink gives that for free.What:
kitty/fish-config-watcher.py→scripts/kitty-fish-config-watcher.py(renamed to carry thekittyprefix); removed the now-emptykitty/directory.kitty-logging installnow symlinks the watcher into the Kitty config dir (ln -sfn) instead of copying — it always tracks the canonical source, so the staleness comparison is gone.kitty-logging uninstallnow removes a symlink (including a dangling one) or a legacy copied file.fish-config-watcher.py, so existingkitty.confmanaged blocks keep working; installs migrate on the nextinstall.README.md,docs/fish-config.md(SSoT),docs/wiki/7-customization.md, and the script header.Note: A symlink dangles if the fish config dir is absent (e.g. a shared
kitty.confon a non-fish machine) where a copy would survive — a non-issue for this single-machine dotfiles setup.Manual Verification
kitty-logging install, then confirm~/.config/kitty/fish-config-watcher.pyis a symlink (ls -l) pointing at~/.config/fish/scripts/kitty-fish-config-watcher.py.kitty-logging statusreports the watcher installed with the correct version and shows the managed block present.kitty-logging installand confirm the regular file is replaced by the symlink.kitty-logging uninstallremoves both the managed block fromkitty.confand the watcher symlink; re-run confirms idempotent no-op.