Skip to content

fix: undo #88 duplicates that collided with #86/#87 #89

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Gradata merged 1 commit into from
Apr 15, 2026
Merged

fix: undo #88 duplicates that collided with #86/#87 #89

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 0 additions & 7 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -203,13 +203,6 @@ docs/launch-post.md
docs/public-launch-narrative.md
docs/RELEASE-*-DRAFT.md

# Orphaned dirs from S105 splits — cloud/ moved to private
# Gradata/gradata-cloud in #76, sdk/ superseded by the flattened repo
# layout in #65. Left on disk as untracked copies that keep showing
# up in `git status` and sometimes get accidentally re-added.
/cloud/
/sdk/

# Railway config for the cloud API now lives in the private
# gradata-cloud repo at cloud/railway.toml — the root-level copy
# here is a stale leftover from before the split.
Expand Down
12 changes: 0 additions & 12 deletions src/gradata/brain.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -916,18 +916,6 @@ def apply_brain_rules(

result = format_rules_for_prompt(applied)
self._rule_cache.put(cache_key, result)

# Fires the in-memory bus so SessionHistory (integrations/session_history.py)
# can track per-session rule effectiveness — it subscribes to rules.injected
# but the emitter was never wired, leaving compute_effectiveness() dormant.
# Cache hits above skip this intentionally: SessionHistory uses a set so
# replays are idempotent within a session, and the cache is per-scope
# so repeated injects in the same scope yield the same rule set.
if applied:
self.bus.emit("rules.injected", {
"rules": [{"id": a.rule_id} for a in applied],
"task": task,
})
return result

def scoped_rules(
Expand Down
Loading