Skip to content

feat: smart filtering with rate limits (DOJ-2433)#8

Open
dbejarano820 wants to merge 1 commit intomainfrom
daniel/doj-2433-smart-filtering
Open

feat: smart filtering with rate limits (DOJ-2433)#8
dbejarano820 wants to merge 1 commit intomainfrom
daniel/doj-2433-smart-filtering

Conversation

@dbejarano820
Copy link
Contributor

Summary

  • Skip trivial commands (cd, ls, pwd, clear, etc.)
  • Rate limit: 30s minimum between teaching triggers
  • Session cap: max 12 insights per session
  • First-ever concepts bypass rate limit (not session cap)
  • Session state in ~/.code-sensei/session-state.json
  • Cleanup on session end

Test plan

  • bash -n passes on all modified scripts
  • Trivial commands produce empty output
  • Rate limiting prevents rapid-fire triggers
  • Session state file created and cleaned up

Closes DOJ-2433

🤖 Generated with Claude Code

Skip teaching triggers for trivial commands (cd, ls, pwd, etc.).
Rate limit: 30s minimum between triggers, 12 per session max.
First-ever concepts bypass rate limit but not session cap.
Session state tracked in ~/.code-sensei/session-state.json.

Closes DOJ-2433

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dbejarano820
Copy link
Contributor Author

🤖 This PR has had no activity for 1+ days.

If it's still needed, please:

  • Resolve any conflicts
  • Address review feedback
  • Push an update

If it's no longer needed, please close it.

— Dojito (automated PR gardener)

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