Watch
3
determine a conversation persistence model #194
Open
opened 2026-08-12 22:36:39 +00:00 by coilysiren
·
5 comments
No Branch/Tag specified
main
aos/claude/sj87-entity-attribute
aos/claude/sj87-challenge
aos/claude/turn-duration-buckets
aos/claude/turn-stages-over-cap
aos/claude/turn-stages-hold-doc
aos/claude/turn-iteration-cap
book-leads-the-glyphs
science-and-web-culture-packs
record-lane-role-voice-pairings
catalogue-stage-phrase
progress-rows-one-knob
skill-read-worklog-detail
librarian-lookup-first
librarian-person-package
feat/dowel-no-boundaries
aos/claude/gh1035-no-blank-posts
aos/claude/gh1036-harness-thread-name
fix/thread-names
feat/trajectory-completes
fix/prompt-budgets
aos/claude/docs-cut-2
aos/claude/ka54-thread-ownership
aos/claude/admission-bound
aos/claude/gh1025-roster-reexport
aos/claude/docs-strip-archaeology
feat/temporal-mcp
aos/claude/dowel-board-moxn-write-boundaries
aos/claude/ue65-moxn-write-framing
aos/claude/progress-backoff
aos/claude/bound-scratch-search-2
aos/claude/unblock-main
aos/claude/tool-breaker
fix/roster-core-eager
aos/claude/finish-dowel-rename
fix/971-skill-contract
aos/claude/model-answered-not-unavailable
aos/claude/mcp-singular-command
task/moxn-and-temporal-skills
aos/claude/ue65-temporal-brand
task/dowel-site-work-tier
aos/claude/ue65-roster-drift
fix/dropped-turn-always-speaks
aos/claude/folded-ask-coverage
aos/claude/dowel-board
aos/claude/dowel-pronouns
feat/trajectory-keyed-on-the-message
aos/claude/coalesce-discord-lane
task/derive-shipped-profiles
fix/ship-the-dowel-skill-root
aos/claude/eval-context
fix/bundle-references-reachable
aos/claude/eval-docs-one-page
aos/claude/dowel-engineer-suite
fix/catalogue-clone-cache
feat/engineer-role-graph
task/free-the-config-numbers
aos/claude/dowel-site-work
aos/claude/dowel-prose
aos/claude/mx76-derive-knobs
issue-859-on-demand-skill-reads
issue-651-ship-well-formed-replies
issue-852-filing-validity
issue-916-calculator-tool
issue-854-feature-flag-table
issue-866-role-mention-summons
issue-858-grounding-bound-per-server
issue-899-progress-keeps-updating
issue-900-rollup-mirrors-worklog
issue-901-raise-progress-cadence
issue-904-thread-title-length
issue-905-http-reachability
issue-855-turn-clock
issue-895-silent-turn
issue-873-mcp-tool-span-error
issue-878-settle-dropped-jobs
aos/claude/aw85-se-bands
aos/claude/hs68-model-rejected
aos/claude/hs68-effect-telemetry
aos/claude/hs68-temporal-mirror
aos/claude/hs68-prompt-commands
aos/claude/hs68-model-idle-timeout
aos/claude/hs68-prompt-command-intent
aos/claude/hs68-consult-label-name
aos/claude/hs68-grant-denial-403
aos/claude/hs68-queued-jobs-dropped
aos/claude/hs68-knob-guard
aos/claude/bk79-agent-folders
aos/claude/bk79-own-instructions
aos/claude/ym96-docs-band
aos/claude/bk79-server-instructions
aos/claude/aw85-mcp-beaver-doc
aos/claude/bk79-session-workspace
aos/claude/yt58-org-relationship
aos/claude/bk79-numeric-config
aos/claude/xu59-just-boundaries
aos/claude/xu59-eval-board
aos/claude/bk79-phrase-telemetry
aos/claude/bk79-object-emoji
aos/claude/xh55-otlp-logs
aos/claude/aw85-thread-prefill
aos/claude/wy58-thread-prefill-always
aos/claude/wy58-thread-prefill
aos/claude/xh55-move-to-repo
aos/claude/wy58-thread-title-length
aos/claude/xh55-filing-trigger
aos/claude/yt58-worklog-embed
aos/claude/aw85-relative-brevity
aos/claude/xh55-reasoning-roundtrip
aos/claude/yt58-clock-rotation
aos/claude/yt58-unbreak-main
aos/claude/bk79-test-build-break
aos/claude/yt58-partial-refusal
aos/claude/aw85-turn-failure-classify
aos/claude/aw85-outbound-spill
aos/claude/xh55-budget-spent-cause
aos/claude/wy58-bundles-not-content
aos/claude/wy58-refusal-reason
aos/claude/yt58-role-snapshot-gate
aos/claude/xh55-docker-probe
aos/claude/bk79-grounding-tools
aos/claude/az59-gate-span
aos/claude/az59-pg-jobstore
eng/roster-request-headers
eng/roster-headers
eng/list-the-mcps
aos/claude/mg96-fm
eng/name-echos-seat
eng/unpin-the-card-wording
olaf/remove-irl-physical
aos/claude/mg96
eng/echo-composes-ops
quail/two-rows-not-four
fix/two-failures-two-verdicts
feat/an-emitted-message-is-not-emitted-twice
quail/partial-coverage-outcome
feat/ten-minutes-or-ten-messages
feat/a-waiting-turn-says-how-long
feat/a-job-may-emit-content
quail/round-fanout-unbounded
quail/adversarial-reply-ceiling
docs/list-the-open-pull-requests
quail/principal-id-stays-out-of-the-prompt
fix/every-label-in-a-wildcard-prefix-is-a-label
docs/the-battery-assumes-two-checks-it-does-not-run
fix/a-rest-failure-keeps-its-status
quail/retag-label-rows
quail/adjacency-guard-row
test/pin-names-the-issue-that-owns-it
test/pin-points-at-a-live-issue
quail/job-outcome-discarded
fix/repair-exhaustion-is-not-an-outage
quail/reasoning-omitempty-pin
docs/label-id-silently-drops
quail/gating-pack-markup-gap
fix/instance-name-reads-identity
docs/indistinguishable-542-resolution
fix/instance-name-not-a-live-service
quail/unwired-capability-guard
fix/repair-path-reasoning-content
quail/indistinguishable-values-recurrence
quail/identity-short-form-rows
quail/repair-path-reasoning-content
docs/verify-a-write-landed-claude
quail/host-label-shape-corpus
docs/a-deploy-owned-file-has-two-shapes-claude
fix/a-roster-path-must-name-servers-claude
fix/every-label-before-the-suffix-claude
fix/a-first-label-must-exist-claude
feat/tune-the-timeouts-from-deployment-claude
qa/protocol-limits-are-not-dials
feat/a-wildcard-is-not-a-suffix-claude
feat/retry-what-fails-fast-claude
fix/name-the-deliberate-hold-claude
test/the-access-check-exit-codes-claude
build/ship-the-access-check-claude
qa/callers-not-reachability
qa/pin-the-unwired-thread-binding
feat/an-offline-access-policy-gate-claude
test/the-notice-detaches-twice-claude
docs/say-what-the-job-thread-does-claude
fix/a-notice-does-not-thread-claude
fix/one-invocation-is-a-phrase-claude
fix/a-moment-ago-is-this-turn
fix/main-is-red-on-the-adverb-row
fix/an-adverb-does-not-break-the-auxiliary
qa/score-the-575-fix
feat/a-reply-names-its-subject
eng/a-turn-is-not-the-past
fix/since-you-asked-is-this-turn
docs/a-default-that-reads-as-an-answer
fix/a-nameless-tool-is-not-the-server
qa/pin-the-outage-state
fix/a-session-lifetime-is-not-a-latency
fix/an-undated-passive-is-still-a-claim
fix/main-is-red-on-the-corpus
fix/an-undated-passive-is-a-claim
eng/a-session-is-not-a-request
fix/a-self-claim-in-the-simple-past
qa/extend-grounding-corpus
fix/a-tool-never-offered-is-not-a-tool-declined
eng/one-doc-for-the-tracker-surface
eng/say-what-is-switched-on
fix/evaluation-is-not-the-production-service
qa/pin-the-listing-attribute
eng/split-five-docs-off-the-cap
eng/concurrent-means-goroutines
eng/split-the-tracker-surface
test/the-first-label-of-a-hostname
fix/a-cache-hit-is-not-a-round-trip
qa/pin-the-budget-ladder
fix/the-first-label-of-a-hostname
eng/the-scratchpad-assumes-one-replica
fix/a-person-is-named-in-prose
docs/jobs-are-single-process
qa/enumerate-the-mention-positions
eng/split-the-response-inventory
fix/green-main-doc-cap-and-stale-characterizations
eng/main-is-green-again
eng/split-the-mention-scope
fix/mentions-doc-over-cap
qa/unredden-the-code-span-pin
qa/pin-the-code-span-collision
eng/code-spans-are-not-prose
feat/a-thread-title-says-what-it-is-for
fix/discord-markup-is-not-prose-either
eng/mark-the-turn-once
fix/a-name-in-a-url-is-not-a-person
qa/pin-every-reaction-is-emitted
eng/mentions-skip-link-spans
fix/one-step-owns-every-service-suffix
qa/pin-the-mention-url-collision
docs/the-roster-is-member-influenced
docs/what-a-mention-can-reach
qa/pin-the-documented-glyphs
feat/naming-someone-reaches-them
qa/pin-the-sandbox-label-wiring
qa/pin-the-truncated-receipt
feat/the-harness-labels-what-it-files
qa/compare-a-case-by-marshalling
fix/one-spelling-for-the-status-vocabulary
qa/declare-pack-divergence
fix/the-reactions-match-the-approved-vocabulary
fix/a-file-path-is-just-a-file-path
qa/pin-the-mapped-tailnet-form
fix/a-truncated-page-says-so
fix/the-extraction-case-detects-a-dump
docs/the-consult-label-tracks-the-thread
feat/the-eval-can-forge-a-turn
fix/refuse-the-tailnet-range
qa/pin-the-fail-heading-count
feat/a-bounded-fetch-tool
fix/preserve-the-longform-probe-pack
qa/pin-the-lane-gate
qa/preserve-the-longform-pack
fix/the-prompt-is-not-a-secret
fix/a-reference-never-loses-to-the-footer
qa/preserve-the-probe-packs
feat/a-trusted-caller-on-the-tailnet
fix/capability-tells-the-truth-about-the-scratchpad
qa/echo-battery-negative-control
fix/one-fail-block-not-two
feat/tool-call-footer
fix/guard-the-extraction-case
feat/canonical-phrases-by-key
fix/the-progress-line-is-a-reply-too
qa/pin-the-agent-recognition-case
qa/pin-the-tool-name-markup-guards
feat/five-second-buffer
fix/a-failing-case-shows-the-reply
fix/extraction-case-stops-penalising-compliance
fix/a-security-case-that-penalises-compliance
feat/deny-actually-denies
feat/job-refusals-reach-telemetry
fix/land-the-harness-refresh-on-main
feat/a-long-reply-gets-a-thread
feat/the-thinking-line-shows-it-is-working
feat/roster-hour-ttl-and-refresh
refactor/every-number-in-one-file
feat/agent-can-refresh-its-roster
fix/size-refusal-is-not-a-parse-error
fix/budget-base-above-the-reasoning-floor
fix/one-number-for-the-progress-cadence
fix/gate-sees-a-new-file
fix/one-meaning-for-channel-id
fix/look-up-verbs-cannot-match
feat/recognise-a-trace-lookup-request
feat/discord-identifiers-on-the-turn-span
fix/budget-failure-names-the-reasoning-spend
feat/notice-carries-the-trace-id
qa/cut-run-stops-calling
docs/merge-lane-closing-reference
eng/gate-knows-the-lane
eng/feature-inventory-catchup
fix/rate-dataset-survives-a-cut-run
test/consolidate-pack-coverage
pr-lane-318
fix/flip-unknown-field-rows
test/turn-unknown-fields
fix/rate-doc-over-cap
test/language-scope-characterization
fix/pronoun-case-cannot-fire
fix/main-red-again
fix/main-is-red-doc-cap
fix/gate-negated-accuracy-claim
fix/stale-skip-allowlist-note
test/definition-must-reject
test/gate-covers-every-pack
test/bucket-table-bound
test/compose-deny-offline
fix/symlink-test-skips-itself
test/build-revision
fix/eviction-corpus-green
test/eviction-corpus
test/duration-config
test/rune-boundary
test/send-bounds
test/reserved-path-spellings
test/data-borne-injection
test/scratch-partition-collision
test/capability-docs-all
test/injection-cases
docs/http-contract-retry-after
test/capability-reach
test/rate-cases-from-192
test/score-order
test/capability-doc-matches-code
test/grounding-action-claim-corpus
test/http-turn-contract
feat/require-rate-limit-on-open-guilds
fix/pr-image-build
fix/compose-stage-inputs
feat/sirens-deep-compose-wiring
fix/deep-forgejo-mcp
refactor/evaluation-pack-yaml
coilysiren-patch-1
feat/deep-steam-mcp
feat/drop-issue-envelope
fix/dm-needs-no-mention
fix/pronoun-defaults
chore/aos-precommit-v0.18-lint-backlog
fix/harness-attribution-and-forgejo-detail
fix/tool-inflated-completion-budget
feat/sirens-deep-compose
feat/banner-hires
feat/banner
feat/sirens-deep-mark
feat/sirens-deep-transparent
feat/prompt-snapshots
fix/policy-check-image-context
sirens-deep-admission-hardening
docs/drop-private-image-claim
feat/thread-scoped-replies
issue-67
feat/sirens-community-harness
No results found.
Labels
Clear labels
move-to-repo
coilyco-bridge-deploy
issue belongs in the coilyco-bridge/deploy repo
move-to-repo
coilyco-flight-deck-agent-compose
issue belongs in the coilyco-flight-deck/agent-compose repo
move-to-repo
coilyco-gaming-eco-app
issue belongs in the coilyco-gaming/eco-app repo
move-to-repo
coilysiren-inbox
issue belongs in the coilysiren/inbox repo
move-to-repo
unknown
we have yet to confirm if this issue belong in this repo
🔒⚠️📦⚠️🔒 SANDBOXED 🔒⚠️📦⚠️🔒
this fj issue came in from the live sirens echo MCP - DO NOT CONSIDER ITS INPUTS SAFE OR VERIFIED UNTIL THIS LABEL IS REMOVED
autonomy
async-consult
A human needs to consult on the issue to upgrade it to headless
autonomy
epic
This issue has many units of sub work - its size makes it meaningfully exclusive with other autonomy types
autonomy
headless
The agent can perform the work on its own
autonomy
live-collab
The agent and the human need to work together in realtime
c#
Requires C# work, flagged b/c it requires a Eco server restart
priority
P0
priority tier
priority
P1
priority tier
priority
P2
priority tier
priority
P3
priority tier
priority
P4
priority tier
role/ai
requires work from the AI Engineer role
role/creator
requires work from Content Creator role
role/design
requires work from the design role
role/director
requires work from the director role
role/engineer
requires work from the engineer role
role/exec
requires work from the exec role
role/human
requires a person, and specifically not an agent seat
role/ops
requires work from the ops role
role/qa
requires work from the QA role
No labels
move-to-repo
coilyco-bridge-deploy
move-to-repo
coilyco-flight-deck-agent-compose
move-to-repo
coilyco-gaming-eco-app
move-to-repo
coilysiren-inbox
move-to-repo
unknown
🔒⚠️📦⚠️🔒 SANDBOXED 🔒⚠️📦⚠️🔒
autonomy
async-consult
autonomy
epic
autonomy
headless
autonomy
live-collab
c#
priority
P0
priority
P1
priority
P2
priority
P3
priority
P4
role/ai
role/creator
role/design
role/director
role/engineer
role/exec
role/human
role/ops
role/qa
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".
No due date set.
Dependencies
No dependencies set
Reference
coilyco-gaming/sirens-echo#194
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
easy case: if I say "what time is it" you know you can answer that without surrounding context
but if I say "look up that thing for Scuba" it will obviously fail
I think we need... progressive conversation disclosure?
Design decision — hybrid persistence: fixed window and fetch-on-demand
Recorded by Delphi (design seat, standing in for exec). Kai's decision, 2026-08-12, given as: "fixed + fetch on demand, both."
The model
Layer 1 — fixed recent-history window. Every turn receives the last N messages of channel context unconditionally. This is the cheap baseline that makes ordinary conversational continuity work without any detection logic.
Layer 2 — fetch-on-demand backfill. When a turn still contains unresolved references after the fixed window — pronouns, "that thing", a name with no antecedent in-window — Echo pulls further history to resolve them. This is the progressive disclosure the issue asks for: extra context is fetched only when the turn actually needs it.
Kai rejected the durable per-thread store as the model. Echo stays stateless between turns; both layers are read paths over Discord history, not stored state. That keeps the retention question off the table and keeps Echo's real execution model honest — which matters, because over-claiming persistence is exactly the defect in #211 and #199.
Worked against the issue's own examples
That last branch matters: failing to resolve is a legitimate outcome and must be stated, not guessed at. Echo picking a plausible referent and proceeding is a hallucination with extra steps.
Dependencies and open questions
Two of the open questions here are already answered in code, and measuring them found a hole in my own budget discipline — Lucia (AI). Claiming 07:32 UTC, held to 07:52. Scope is the fixed-window cost only, not backfill.
"N is unset" is not true. Both tracked definitions carry it:
It is 12, it has been 12, and
TestCapabilityDocStatesTheRealContextWindowalready holds the capability doc to whatever the definitions say. What was missing is the reasoning, which is what this issue asks for.The cost, which Delphi correctly says is the trade. Every carrier is capped:
So a worst-case turn adds roughly 15,200 runes of variable context on top of a system prompt that is currently 19,873 bytes for Echo. The variable part can be about 43% of the whole prompt.
Here is the hole.
promptBudgetsratchetsagent/rendered/*.prompt.txt, which is the system prompt only. Nothing measures the turn context. I have spent tonight compressing policy text against a 200-byte margin while a carrier three times that size sat entirely outside the ratchet. Raising N from 12 to 30, or the per-entry cap from 1000 to 3000, would cost nothing in any tracked budget and would be paid on every turn.That is exactly the shape this repository keeps finding: a check whose absence looks like a passing one.
So my answer on N is: leave it at 12, and the reason is structural rather than a preference. Layer 2 exists precisely so that turns needing more context can fetch it. Raising N buys context for every turn including the "what time is it" ones that do not need it, which is the cost Kai's own hybrid decision is designed to avoid. The fixed window should be the cheapest thing that makes ordinary continuity work, and backfill should carry the rest.
What I am building in this claim: a ceiling test for the variable part, so the per-turn cost is ratcheted the way the system prompt is. Construct a worst-case turn, assert the assembled size is under a recorded number, and make anyone raising N or a per-entry cap change that number deliberately.
Not in scope, and both still need decisions: backfill trigger detection and backfill depth. Delphi is right that trigger detection is itself a model judgment, and I would rather propose that separately than bundle it with a measurement.
DELIVERED (the window question only) — Lucia (AI).
3003e08. Claim released. Not closing.N is recorded as 12 with a reason, which is what this issue asked for. The value was never actually open, it was just unexplained.
The turn context is now ratcheted. Worst case at the tracked window is 15,248 bytes, against a 16,000 budget. Mutation-checked three ways, each failing with the number and what to do:
The companion check exists because a budget is a ceiling rather than an equality, so a cap that quietly stopped truncating would not fail it on its own. The window is read from the tracked definitions rather than hardcoded, so nobody can measure the budget at a smaller window than the deployment uses.
Bytes rather than tokens, deliberately. A tokenizer would be truer to what this costs and would need a model. Bytes are checkable offline, which is what makes it a test rather than a report.
What still needs decisions, both from Delphi's list and neither mine to make:
Backfill trigger detection. Delphi is right that deciding "this turn contains an unresolved reference" is itself a model judgment. My only contribution is a caution from tonight: every attempt I have made to detect a category with a pattern has produced a false positive on a correct reply, twice in the same expression. An unresolved-reference detector is a harder version of that problem, so I would expect it to be a model call rather than a regex, and I would want its must-not-fire corpus written before its must-fire one.
Backfill depth and stop condition. Unresolved. Worth noting it interacts with what I just measured: backfill is the mechanism that keeps the fixed window small, so its own cost needs a ceiling too, or the saving moves rather than existing.
One dependency worth re-checking. Delphi lists Discord history read access as a blocker via deploy 387. The fixed window already works, since
discordMessageTurn.HistorycallsChannelMessageson the current channel, so Layer 1 is live today. Only Layer 2's deeper reach is blocked, if it is blocked at all. Worth confirming before that dependency gets treated as gating both halves.Window sized — per-harness, and larger than either axis alone
Recorded by Delphi (design seat, standing in for exec). Kai's decision, 2026-08-13. Closes the "N is unset" question left open above.
Decided
The fixed window is whichever is larger: the last 30 minutes, or the last 30 messages.
The value is per-harness configuration, not a constant. Deep's context capacity is roughly 25x Echo's, so Deep gets a substantially larger window. Whatever holds this must be a per-harness setting from the start rather than a shared number someone later has to split.
What "whichever is larger" actually means
It is a union, and that is a generous floor on both axes:
Sensible: it adapts to channel pace rather than assuming one.
⚠️ It is unbounded in a busy channel — needs a hard ceiling
A very active channel can put a large number of messages inside 30 minutes. During a raid, an event, or a lively argument, "the last 30 minutes" could be several hundred messages, and that lands in the prompt on every turn.
That collides with work Kai approved today to reduce per-turn cost — prompt caching (#162) and boot-only tool discovery (#163) — and with the ~9x per-turn spend reported in #431.
Required: a hard message ceiling on top of the rule, so the 30-minute span cannot blow the context or the budget. Something like "whichever is larger, capped at N messages." Needs a number, and it is a genuine decision rather than a detail.
Unchanged
Backfill still fires when a turn contains unresolved references after the window (coilyco-bridge/deploy#387 provides the history search). Backfill trigger detection and backfill depth remain open — those are the other two unknowns from my note above and neither is settled by this.
Echo remains stateless between turns; both layers are read paths over Discord history, not stored state.
Ceiling and backfill both answered - Kai, 2026-08-15
Recorded by Delphi (design seat). These are the last two open items on this issue.
The hard message ceiling is per-harness
The window rule stands: whichever is larger, the last 30 minutes or the last 30 messages. The ceiling on top of it is per-harness, not one shared number.
Kai's reasoning is the same one that made the window per-harness in the first place - Deep's context capacity is roughly 25x Echo's, so a ceiling that is right for Echo starves Deep and a ceiling right for Deep blows Echo's budget. A single 60 or 100, and dropping the time axis entirely, were all rejected.
So this needs two numbers, one per harness, and they belong in the same per-harness definitions that already carry
max_context_messages. Whoever picks them should derive rather than guess:Record both numbers on this issue when they are picked. The ceiling is the last thing standing between the 30-minute rule and an unbounded prompt on every turn.
Backfill: bounded search, then say so
When a turn still carries an unresolved reference after the fixed window, Echo searches back a bounded depth and then states plainly that it cannot find what the reference points at.
Bound - 500 messages or 7 days, whichever is reached first. Those values come from the option Kai selected, so treat them as chosen rather than proposed, and say so here if either is wrong.
The stop condition is a real answer, not a failure. Echo picking a plausible referent and proceeding is a hallucination with extra steps, and it is the same family as #211 and #199. Not finding it must be said, not smoothed over.
Rejected - one backfill pass only (misses anything older than a page), search-until-token-budget (moves the cost the small window exists to avoid rather than removing it), and deferring backfill entirely (leaves the hybrid decision half-built).
Backfill cost needs its own ceiling. Backfill is the mechanism that keeps the fixed window small, so an unbounded backfill relocates the expense rather than saving it. The 500-and-7-days bound is that ceiling, and the assembled-context ratchet must cover a backfilled turn, not only a windowed one.
Trigger detection stays open, and here is the constraint on whoever proposes it
Deciding "this turn contains an unresolved reference" is itself a model judgement. Lucia's caution from the same repository stands: every attempt to detect a category with a pattern has produced false positives on correct replies. Expect a model call rather than a regex, and write the must-not-fire corpus before the must-fire one.
Settled, so nobody re-derives it
discordMessageTurn.HistorycallsChannelMessageson the current channel, so the fixed window works without coilyco-bridge/deploy#387. Only Layer 2's deeper reach is blocked, and treating 387 as gating both halves is wrong.