feat: cross-platform alias enrichment via JSONL#83
Merged
Conversation
Add support for operator-curated state/people.jsonl and state/channels.jsonl files that enrich the auto-populated phone book with cross-platform aliases (Bluesky, email, Google Docs, nicknames). The merged aliases are injected as a [PEOPLE] + [CHANNELS] block in every turn prompt, giving agents always-visible context about who is who across platforms — structurally preventing attribution errors from mismatched display names. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Owner
|
@strix-tkellogg we need a better process in here for upgrading their current phone book into jsonl so it doesn't crap out |
Addresses Tim's feedback on PR #83 — existing operators need a smooth upgrade from auto-populated phone books to JSONL enrichment. - `export_to_jsonl()`: generates starter people.jsonl + channels.jsonl from existing phone-book.md with placeholder fields for cross-platform aliases. Never overwrites existing files. - `open-strix export-phone-book` CLI command for operators to run the migration before adding Bluesky/email/docs aliases manually. - Logging in `_load_jsonl()`: warns on malformed JSON lines with line numbers instead of silently skipping. - 4 new tests (export, skip-existing, roundtrip, malformed-warning). 49 total pass. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Collaborator
Author
|
Good call — pushed a fix (56d4d3b). What's new:
Upgrade path for existing operators: open-strix export-phone-book --home /path/to/agent
# Edit the generated JSONL files to add cross-platform aliases
# Restart — enrichment happens automatically at startup4 new tests + roundtrip test (export → enrich from exported files). 214 pass, 1 skipped. |
jptreen
pushed a commit
to jptreen/open-strix
that referenced
this pull request
Apr 11, 2026
Merged upstream changes: - PR tkellogg#80: Cycle detection prompts reflection instead of just stopping - PR tkellogg#81: Five-whys integration into harness failure paths - PR tkellogg#82: Five-whys chainlink docs - PR tkellogg#83: Phone book → JSONL migration + alias enrichment Conflict resolution: - app.py: Kept both _withhold_final_text (ours) and _last_turn_failure (upstream) - prompts.py: Preserved auto-send model while adding aliases/failure sections - tools.py: Added missing file tools to tools list (read_file, glob_files, edit_file, write_file) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
state/people.jsonlandstate/channels.jsonlsupport for operator-curated cross-platform aliases (Bluesky, email, Google Docs names, channel nicknames)[PEOPLE]+[CHANNELS]block in every turn prompt — agents always see who is who across platformsHow it works
enrich_from_jsonl()merges JSONL records into phone book entries by Discord IDrender_aliases_block()formats a compact reference blockrender_turn_prompt()includes the block as section 6Files changed
phone_book.py— JSONL loading, enrichment, alias renderingconfig.py—people_jsonl/channels_jsonlpaths onRepoLayoutapp.py— enrichment at startup, aliases in promptprompts.py—aliases_blockparameter, system prompt updatetests/test_phone_book.py— 13 new tests (45 total, all passing)Test plan
🤖 Generated with Claude Code