fix: de-hardcode the author's DA name across TOOLS, PULSE, and hooks#1457
Open
anikin-xyz wants to merge 1 commit into
Open
fix: de-hardcode the author's DA name across TOOLS, PULSE, and hooks#1457anikin-xyz wants to merge 1 commit into
anikin-xyz wants to merge 1 commit into
Conversation
The author's DA name (kai) was baked into ~15 sites that should read the
configured DA name so a fresh install reflects the name the principal chose
during setup.
- TOOLS (TelosFreshness, DAGrowth, DASchedule, MemoryStatus, MemoryInsights,
WorkSweep) + hooks/ReminderRouter: read via getDAName() (hooks/lib/identity).
- PULSE modules (work.ts, telegram.ts): resolve the DA name through the same
identity/LifeosConfig sources already used in those files; the Agent:<da>
label producers now share one source so their labels agree.
- BannerRetro: render the DA name (stats.name, defaults to "LifeOS") instead
of hardcoded "KAI" block-letter art.
- MemoryGraph: drop personal names ("kai", "daniel") from the stopword list.
- PULSE.toml [da].primary: neutral placeholder + pointer to the canonical
LIFEOS_CONFIG.toml [da].name.
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
1 participant
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.
Following the invitation in #1334 to help scrub the author's personal identifiers out of the shipped payload, this de-hardcodes the DA name (
"kai") so a fresh install reflects the name the principal chose during setup instead of the author's.One root cause — the DA name literal baked into ~15 sites — across six clusters:
TelosFreshness.ts(by = "kai"onbumpTelosTimestamp/bumpContextTimestamp).[da].primary— personal literal → neutral placeholder + pointer to the canonical config.DAGrowth.ts,DASchedule.ts(?? "kai", help text).MemoryStatus.ts(kai status),MemoryInsights.ts(kai insights),BannerRetro.ts(KAIblock-letter ASCII).WorkSweep.ts(×4),PULSE/modules/work.ts,hooks/ReminderRouter.hook.ts(Agent:kai→Agent:<da-name>).telegram.tsspeaker-label regex and voice-note filename,MemoryGraph.tsstopword list (drops the personal nameskai,daniel).Mechanism: TOOLS and hooks read the configured name via
getDAName()(hooks/lib/identity.ts); PULSE modules resolve it through the config they already load. The threeAgent:<da>label producers share one source so their labels agree.BannerRetrorenders the DA name (stats.name, default"LifeOS") instead of hardcoded art. Surgical — no behavior change beyond the name value.Deferred as a follow-up (template-semantic, riskier than a surgical rename): the structural
kaikeys inskills/Prompting/Templates/Data/Agents.yaml/VoicePresets.yaml— the displayed names there already use{{DA_NAME}}; only the lookup keys are literal.