Skip to content

fix: rename "Hooks" to "Policies" in UI and CLI output#9

Merged
NiveditJain merged 2 commits into
mainfrom
ef-11
Apr 6, 2026
Merged

fix: rename "Hooks" to "Policies" in UI and CLI output#9
NiveditJain merged 2 commits into
mainfrom
ef-11

Conversation

@NiveditJain

@NiveditJain NiveditJain commented Apr 6, 2026

Copy link
Copy Markdown
Member

Summary

  • Updates display strings in app/policies/hooks-client.tsx to say "Policies are not installed" / "Policies installed" / "Install policies"
  • Updates src/hooks/manager.ts CLI output to say "Failproof AI Policies" and "Policies not installed"

Test plan

  • Visit the Policies tab — status indicator should read "Policies installed" / "Policies not installed"
  • Visit the Activity tab with no policies installed — empty state should read "Policies are not installed"
  • Run failproofai --list with no hooks installed — output should read "Failproof AI Policies — not installed"

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated user-facing text across the Activity, Policies tabs, and CLI to consistently refer to "policies" instead of "hooks" when describing installation and monitoring status.

Updates display strings in hooks-client.tsx and manager.ts to use
"Policies" terminology consistently, matching the installed feature name.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Apr 6, 2026

Copy link
Copy Markdown
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: d2e41796-f588-4319-b8c2-5493f1203cb4

📥 Commits

Reviewing files that changed from the base of the PR and between 8dd8714 and 09d4955.

📒 Files selected for processing (1)
  • __tests__/hooks/manager.test.ts
✅ Files skipped from review due to trivial changes (1)
  • tests/hooks/manager.test.ts

📝 Walkthrough

Walkthrough

User-facing strings updated to replace “hooks” terminology with “policies” across frontend UI and CLI output; no functional or control-flow changes.

Changes

Cohort / File(s) Summary
UI Text Updates
app/policies/hooks-client.tsx
Activity and Policies tabs: empty-state messaging, installation-warning string, status labels, and primary button text changed from "hooks" to "policies".
CLI Output Text
src/hooks/manager.ts
listHooks CLI output header and follow-up instruction updated to use "Policies — not installed" / "Policies not installed".
Tests
__tests__/hooks/manager.test.ts
Updated test expectation string to assert "Policies — not installed" instead of the former "hooks" phrasing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nibbled through strings with delight,

Replaced "hooks" with "policies" — tidy and right.
No logic disturbed, just a lexical spree,
The UI now reads how it ought to be. 🥕


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

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

The "Hooks not installed" string was renamed to "Policies — not installed"
in the source, but the test assertion wasn't updated to match.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@NiveditJain NiveditJain merged commit 3d8f45f into main Apr 6, 2026
8 checks passed
@NiveditJain NiveditJain deleted the ef-11 branch April 21, 2026 01:32
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.

1 participant