Skip to content

Conversation

@celdrake
Copy link
Collaborator

@celdrake celdrake commented Jan 23, 2026

Remove the library since it's not being used.

Summary by CodeRabbit

  • Chores
    • Removed the Ansible library and its workspace entry from the monorepo.
    • Removed the library’s public exports, UI components, navigation, fetch/API utilities, styles, and TypeScript config.
    • Updated build scripts to exclude the removed library.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 23, 2026

Walkthrough

The PR removes the entire libs/ansible workspace from the monorepo, including its package configuration, TypeScript settings, source components, hooks, utilities, and constants. The root package.json is updated to remove the workspace entry and its build step.

Changes

Cohort / File(s) Summary
Workspace & Manifests
package.json, libs/ansible/package.json, libs/ansible/.npmignore, libs/ansible/tsconfig.json
Removed ./libs/ansible from root workspaces and build script. Deleted ansible package manifest, .npmignore, and tsconfig.
Public API / Exports
libs/ansible/src/index.ts
Removed re-exports of FctlAppContext and DevicesPage from the ansible package entry.
Context & Providers
libs/ansible/src/components/AppContext/AppContext.tsx
Deleted AppContext provider, useValuesAppContext hook, and FctlAppContext wrapper component.
Presentational Components
libs/ansible/src/components/ScrollablePage/ScrollablePage.tsx, libs/ansible/src/components/ScrollablePage/ScrollablePage.css
Deleted ScrollablePage component and its CSS rule (.fctl-scrollable-page).
Routing / Navigation
libs/ansible/src/const.ts, libs/ansible/src/hooks/useFctlNavigation.tsx
Removed appRoutes route mappings and the useFctlNavigation hook that constructed the navigation/route tree.
Networking Utilities
libs/ansible/src/hooks/useFetch.ts, libs/ansible/src/utils/apiCalls.ts
Deleted useFetch hook and all HTTP helper utilities (fetchUiProxy, fetchData, postData, putData, deleteData, patchData) including CSRF header handling and WebSocket endpoint helper.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~30 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Remove Ansible lib' directly and accurately describes the main objective of the pull request—removing the unused Ansible library.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

@celdrake celdrake requested a review from rawagner January 23, 2026 12:54
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.

2 participants