Skip to content

chore(gastown): promote gastown-staging to main#2974

Open
jrf0110 wants to merge 2 commits intomainfrom
gastown-staging
Open

chore(gastown): promote gastown-staging to main#2974
jrf0110 wants to merge 2 commits intomainfrom
gastown-staging

Conversation

@jrf0110
Copy link
Copy Markdown
Contributor

@jrf0110 jrf0110 commented Apr 30, 2026

Summary

Single-commit batch from `gastown-staging` carrying a local-dev binding fix that was missed from the previous promotion:

  • fix(gastown): point dev `GIT_TOKEN_SERVICE` binding at `git-token-service-dev` — `services/git-token-service/wrangler.jsonc` overrides its worker name to `git-token-service-dev` in `env.dev`, but gastown's `env.dev.services` binding still referenced the base `git-token-service` name. Wrangler's local dev registry does exact-name matching, so the binding showed `[not connected]` whenever both workers ran side by side. Every other consumer in the repo (`cloud-agent-next`, `security-sync`, `security-auto-analysis`) already uses `git-token-service-dev` in their `env.dev` block — gastown was the outlier.

Production binding (top-level `services`) is untouched; only the `env.dev` block was changed.

Verification

  • Manually restarted `wrangler dev --env dev` for both `services/git-token-service` and `services/gastown` locally and confirmed the gastown binding now shows as connected instead of `[not connected]`.

Visual Changes

N/A (wrangler config only).

Reviewer Notes

  • One-line change in `services/gastown/wrangler.jsonc` `env.dev.services[0].service`. No runtime code touched, no impact on the deployed production worker.

…e-dev

git-token-service's wrangler env.dev overrides the worker name to
'git-token-service-dev', but gastown's env.dev.services binding was
still referencing the base 'git-token-service' name. Wrangler's local
dev registry does exact-name matching, so the binding showed as
[not connected] whenever both workers were running side by side.

Every other consumer in the repo (cloud-agent-next, security-sync,
security-auto-analysis) already uses 'git-token-service-dev' in their
env.dev block; gastown was the outlier.
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Apr 30, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • services/gastown/container/src/process-manager.ts
  • services/gastown/container/src/process-manager.test.ts
  • services/gastown/container/vitest.config.ts

Reviewed by gpt-5.5-20260423 · 783,937 tokens

…2999)

When a user changes the mayor's model in town settings, updateAgentModel
restarts the SDK server with new KILO_CONFIG_CONTENT and resumes the
existing session from kilo.db. Commit 9785570 intentionally stopped
sending any session.prompt on resume to avoid duplicating the
MAYOR_STARTUP_PROMPT, but that also dropped the model param — so the
resumed session kept its prior per-session model until the user ran
/model manually.

Extract the fresh vs. resumed session-prompt logic into applyModelToSession
and on resume send a noReply:true prompt carrying only the new model
param. This updates the SDK server's per-session model without replaying
the startup prompt. Errors on the resume path are swallowed so the
hot-swap still succeeds; the SDK server fell back to the config-loaded
model at startup, which was already updated.

Add container tests covering both fresh and resumed paths.

Co-authored-by: John Fawcett <john@kilcoode.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant