Rename the /sigint slash command + adopt tenant-chart-base#5
Merged
Conversation
The repo was already literal-named everywhere internally; the only codename left
was the Slack slash command. Greenfield (not deployed), so this finishes the
rename and lifts the chart onto the shared library.
─────────────────────────── codename ───────────────────────────
The Slack slash command /sigint becomes /competitive-intelligence (commands.ts
registration + help text, the CLI banner in display.ts, and the doc references in
README/CLAUDE/AGENTS/ARCHITECTURE). The stale "the slash command stays /sigint"
invariant notes are reworded — everything is the literal name now. The default
alert channel #competitive-intel stays (a short handle, not a codename), as does
the POSIX SIGINT signal handler.
─────────────────────────── chart retrofit ───────────────────────────
The chart vendors tenant-chart-base under chart/charts/ + declares it as a
file:// dependency. serviceaccount/networkpolicy/grafana-dashboard collapse to
one-line `{{ include "tenant-chart-base.*" . }}` wrappers; prometheusrule keeps
its app-specific alert groups but uses the library helpers; deployment/service/
externalsecret reference the library helpers; the local _helpers.tpl is gone.
values.yaml gains a structured otel.resourceAttributes map (the library labels
read it for agents.nanohype.dev/{tenant,platform}).
Rendered output is faithful — the only diffs vs before are label values now
quoted (same string) and dropped inline comments. helm lint clean; typecheck +
50 tests pass.
stxkxs
force-pushed
the
drop-sigint-adopt-tenant-chart-base
branch
from
3934fbb to
2005091
Compare
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.
The repo was already literal-named everywhere internally; the only codename left was the Slack slash command. Greenfield (not deployed), so this finishes the rename and lifts the chart onto the shared library.
Codename:
/sigint→/competitive-intelligence(command registration + help text, the CLI banner, and the doc references). The stale "slash command stays /sigint" invariant notes are reworded.#competitive-intel(a short channel handle, not a codename) and the POSIXSIGINTsignal handler stay.Chart retrofit: vendors
tenant-chart-baseunderchart/charts/+ afile://dependency. serviceaccount/networkpolicy/grafana-dashboard collapse to one-line{{ include "tenant-chart-base.*" . }}wrappers; prometheusrule keeps its app-specific alerts but uses the library helpers; deployment/service/externalsecret use the library helpers; local_helpers.tplis gone;values.yamlgains a structuredotel.resourceAttributesmap.Faithful: old-vs-new
helm templatediff shows only label values now quoted (same string) + dropped inline comments. helm lint clean; typecheck + 50 tests pass.First of the four greenfield codename-rename + chart-retrofit PRs.