Clean up stray legacy PAI/Kai branding remnants (cosmetic only)#1582
Clean up stray legacy PAI/Kai branding remnants (cosmetic only)#1582elhoim wants to merge 4 commits into
Conversation
…ding in internal function name)
…eos* (unreferenced stale type identifiers)
|
Thanks — this is exactly the kind of PR I want. You verified each candidate against the repo instead of blind find-and-replace, and every call in the "found but NOT changed" list was right, especially leaving the dual-brand regexes in SecurityFilter alone. One quirk of this repo: the public tree is a generated artifact, rsync'd from my private source at release time, so anything merged only here gets wiped on the next release. The way I adopt community changes is to port them into source with credit, which I've done. All four of your changes are in as-is and will ship with the next release. Good catch on DaemonAggregator too (the internal-architecture filter matches Closing because it's ported, not because it was rejected. Thanks again. |
Summary
Cleans up a small number of stray legacy "PAI"/"Kai" branding remnants in the skills — verified individually against this repo's own content, not a find-and-replace.
Each candidate string was classified as either a cosmetic remnant (safe to modernize — prose/identifier with no functional effect) or a functional identifier (a real path, trigger phrase, or flow name that code/users actually depend on). Only the cosmetic ones are changed here.
Changes
LifeOS/install/skills/Art/Tools/Generate.tsstampKaiSignature→stampDaSignature{{DA_NAME}}template — only the internal function name still said "Kai".LifeOS/install/skills/Remotion/Tools/Theme.tsPAITheme/PAIColors/PAITypography/PAIAnimation→LifeosTheme/LifeosColors/LifeosTypography/LifeosAnimationPai*→Lifeos*identifier sweep, which appears to have missed this file.LifeOS/install/skills/CreateCLI/SKILL.md--repo pai→--repo myrepoLifeOS/install/skills/LifeOS/install/settings.enhancements.json"...runs from PAI/LIFEOS_StatusLine.sh."→"...runs from LIFEOS/LIFEOS_StatusLine.sh."LifeOS/Tools/DeployComponents.ts) useLIFEOS/throughout with zeroPAIreferences, so the fact told to the DA no longer matched reality. This file is JSON, not.md, so it fell outside the repo's earlier "prose branding" doc sweep.Found but intentionally NOT changed — why
These all matched "PAI"/"Kai" but are load-bearing, not cosmetic:
Knowledge/SKILL.md— referencespai-knowledge-linking-architecture.md. This file also appears to be missing/dangling already; renaming the reference wouldn't fix that, and could make the drift harder to spot. Left for a maintainer to resolve directly.Interceptor/SKILL.md—/tmp/pai-screenshots/. Verified functional:install/hooks/ContextReduction.hook.shliterally doesmkdir -p /tmp/pai-screenshots && cd's there before rewriting Interceptor--savecalls. Renaming the doc without touching the hook would break the documented behavior.Upgrade/SKILL.md—"pai upgrade"in theUSE WHENtrigger list. A real, user-typed trigger phrase; removing it silently drops a working trigger.CreateSkill/Workflows/CreateSkill.mdandUpdateSkill.md—`UpdateKaiRepo`ship-flow name, invoked by the (templated-out) private release skill at version-bump time. A real flow name, not prose.Daemon/Tools/SecurityFilter.ts—(?:LIFEOS|PAI)/USER/,(?:LIFEOS|PAI)/MEMORY/,PAI/Algorithm/v[\d.]+\.mdregexes. Deliberate dual-branding patterns in the public-daemon-profile security scrubber — removingPAInarrows what gets redacted. Security-relevant, not cosmetic.Daemon/Tools/DaemonAggregator.ts—thesis.match(/PAI\/|hooks\/|.../)gates which knowledge ideas get treated as "internal architecture" and skipped. Changing the pattern changes filtering behavior; out of scope for a branding-only PR (possibly worth its own look — it doesn't also matchLIFEOS/).LifeOS/Tools/InstallEngine.ts— comments describing detection of "a prior LifeOS/PAI install". These correctly document real backward-compatibility logic for legacy installs; not a remnant.No filesystem paths, usernames, or credentials from any local environment are referenced anywhere in this diff.