Skip to content

Moved config folder location + added uninstallation options - #12

Merged
jamius19 merged 4 commits into
mainfrom
feature/move-config-folder
Aug 1, 2026
Merged

Moved config folder location + added uninstallation options#12
jamius19 merged 4 commits into
mainfrom
feature/move-config-folder

Conversation

@jamius19

@jamius19 jamius19 commented Aug 1, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features

    • Added an uninstall command for removing Sibyl integrations from supported tools and custom files.
    • Added cleanup for imported instructions, standalone documents, configuration references, and embedded Sibyl sections.
    • Added validation and safe handling for malformed configuration or instruction markers.
  • Bug Fixes

    • Updated configuration and plugin locations to use the standard ~/.config/sibyl directory.
    • Legacy plugin locations are no longer loaded.
  • Documentation

    • Documented uninstall behavior, supported targets, configuration paths, and setup architecture.
    • Updated the release version to 0.1.2.

@jamius19 jamius19 self-assigned this Aug 1, 2026
@jamius19 jamius19 added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 1, 2026
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 63b4b393-1b8c-420a-8ac6-a7dd4b29aeb9

📥 Commits

Reviewing files that changed from the base of the PR and between 25048d5 and f2b756d.

📒 Files selected for processing (15)
  • AGENTS.md
  • README.md
  • docs/CONFIGURATION.md
  • docs/CREATING-PLUGINS.md
  • package.json
  • src/cli.test.ts
  • src/cli.ts
  • src/plugin-loader.test.ts
  • src/plugin-loader.ts
  • src/plugins/builtin-ai-ask/main.test.ts
  • src/plugins/builtin-ai-ask/main.ts
  • src/setup-command.test.ts
  • src/setup-command.ts
  • src/setup.test.ts
  • src/setup.ts

📝 Walkthrough

Walkthrough

The CLI now supports uninstall for removing Sibyl instruction integrations. Configuration and plugin paths use ~/.config/sibyl. Help text, documentation, version information, error messages, and tests reflect these changes.

Changes

Uninstall and XDG configuration migration

Layer / File(s) Summary
Migrate configuration and plugin paths
src/setup.ts, src/plugin-loader.ts, src/setup.test.ts, docs/CONFIGURATION.md, docs/CREATING-PLUGINS.md, src/cli.ts, src/cli.test.ts, src/plugins/builtin-ai-ask/*
Configuration and external plugins now use ~/.config/sibyl. Documentation, error messages, and tests use the new paths.
Implement uninstall cleanup
src/setup-command.ts, src/setup-command.test.ts
runUninstall removes Claude, OpenCode, embedded, and custom-file integrations. It validates targets and JSON, preserves malformed blocks, supports repeated targets, and stops after failures.
Expose and document the uninstall command
src/cli.ts, src/cli.test.ts, README.md, AGENTS.md, package.json
The CLI dispatches uninstall, reports version 0.1.2, and documents targets, cleanup behavior, and usage examples.

Estimated code review effort: 4 (Complex) | ~45 minutes

Sequence Diagram(s)

sequenceDiagram
  participant CLI
  participant runUninstall
  participant HostFiles
  participant OpenCodeConfig
  CLI->>runUninstall: pass uninstall targets
  runUninstall->>HostFiles: remove imports and embedded blocks
  runUninstall->>OpenCodeConfig: validate JSON and remove instruction references
  OpenCodeConfig-->>runUninstall: return updated configuration
  runUninstall-->>CLI: report completion or fail-fast error
Loading

Possibly related PRs

  • postapsis/sibyl#2: Updates shared AGENTS.md, CLI, setup, and plugin-loader documentation and tests.
  • postapsis/sibyl#9: Introduced the builtin-ai-ask plugin updated by this change.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/move-config-folder

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jamius19
jamius19 merged commit 4cd0ba8 into main Aug 1, 2026
1 of 2 checks passed
@codecov

codecov Bot commented Aug 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.56627% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/setup-command.ts 90.14% 2 Missing and 5 partials ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant