Skip to content

Conversation

@Snider
Copy link
Owner

@Snider Snider commented Nov 13, 2025

The help package has been moved to its own repository at Snider/help. This commit removes the local copy of the package from this repository and updates the dependencies.

The help package has been moved to its own repository at Snider/help. This commit removes the local copy of the package from this repository and updates the dependencies.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 13, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed in-app help system entirely, including the help service implementation, user interface functionality, documentation content, styling definitions, configuration files, and related build dependencies
    • Removed help service integration from the application runtime layer

Walkthrough

The entire help service has been removed from the codebase. This includes the Go service implementation, test coverage, runtime wiring integration, documentation, build configuration, and styling assets.

Changes

Cohort / File(s) Summary
Help service implementation
pkg/help/help.go, pkg/help/help_test.go
Deleted the complete help service including Service struct, Options type, constructors (New, newHelpService, Register), lifecycle methods (ServiceStartup, HandleIPCEvents), display control methods (Show, ShowAt), and all associated test scaffolding, mocks, and test cases.
Documentation and build tooling
pkg/help/mkdocs.yml, pkg/help/src/index.md, pkg/help/requirements.txt, pkg/help/taskfile.dist.yml
Removed MkDocs configuration, module documentation, Python dependency specification (mkdocs-material), and task runner configuration for development, build, and deployment workflows.
Help styling assets
pkg/help/assets/stylesheets/extra.css, pkg/help/src/stylesheets/extra.css
Deleted all CSS variable declarations (primary colours, slate theme overrides) and stylesheet rules (hero sections, features grid, responsive layouts).
Runtime integration
pkg/runtime/runtime.go, pkg/runtime/runtime_test.go
Removed Help service field from Runtime struct, eliminated help service factory wiring, and removed related test assertions and imports.

Possibly related PRs

Suggested labels

enhancement

Poem

A rabbit hops through cleaner ground,
No help files bundled all around!
The code grows light, untethered free,
Simplicity's our victory! 🐰

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-pkg-help

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8a392c7 and cfd8915.

⛔ Files ignored due to path filters (7)
  • pkg/help/src/images/cross-platform.jpeg is excluded by !**/*.jpeg
  • pkg/help/src/images/decentralised-vpn.jpg is excluded by !**/*.jpg
  • pkg/help/src/images/favicon.ico is excluded by !**/*.ico
  • pkg/help/src/images/illustration.png is excluded by !**/*.png
  • pkg/help/src/images/lethean-logo.png is excluded by !**/*.png
  • pkg/help/src/images/private-transaction-net.png is excluded by !**/*.png
  • pkg/help/src/images/secure-data-storage.jpg is excluded by !**/*.jpg
📒 Files selected for processing (10)
  • pkg/help/assets/stylesheets/extra.css (0 hunks)
  • pkg/help/help.go (0 hunks)
  • pkg/help/help_test.go (0 hunks)
  • pkg/help/mkdocs.yml (0 hunks)
  • pkg/help/requirements.txt (0 hunks)
  • pkg/help/src/index.md (0 hunks)
  • pkg/help/src/stylesheets/extra.css (0 hunks)
  • pkg/help/taskfile.dist.yml (0 hunks)
  • pkg/runtime/runtime.go (1 hunks)
  • pkg/runtime/runtime_test.go (0 hunks)

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 and usage tips.

@Snider Snider merged commit ed5c4a1 into dev Nov 13, 2025
2 of 3 checks passed
@Snider Snider deleted the remove-pkg-help branch November 13, 2025 14:28
@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@coderabbitai coderabbitai bot added the enhancement New feature or request label Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants