Repository navigation
feat(sync_env_clock): an env with no gateway clock, as a computer deployed alone, is refused saying to deploy it in a multi env (FD-5477) - #677
Merged
earakely-scale merged 2 commits intoNov 14, 2026
Conversation
…loyed alone, is refused saying to deploy it in a multi env (FD-5477) Arming an env whose card offers no clock/v1 set_time failed with "env 'desk' does not offer 'set_time' on urn:agentenv:clock/v1". It now says the env has no clock to arm, and that a computer follows a multi env's gateway. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…oy a multi env The arm call raises EnvCapabilityUnsupported for a record with no card or card URL too, an older deploy that has a gateway; both refusals are StepContextErrors, as #632 made the step's other refusals of a task's clock. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
earakely-scale
deleted the
edgararakelyan/fd-5477-hb17-sync_env_clock-computers-follow
branch
November 14, 2026 01:43
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
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.
What changes
FD-5477 (HB1.7). Most of what it scoped already landed with FD-5476 (#649), or turned out not to be needed; this is what was left.
clock/v1set_time, as a computer deployed alone, failed withenv 'desk' does not offer 'set_time' on urn:agentenv:clock/v1. It now says the env has no clock to arm, that a computer deployed alone follows no clock, and to deploy it in a multi env, whose gateway it follows (Q12). The arm call raisesEnvCapabilityUnsupportedfor a record that kept no card or card URL too, an older deploy that has a gateway: that one is told its record keeps no card and to deploy it again, not to deploy a multi env. Both areStepContextErrors, as fix(task_step): sync_env_clock raises StepContextError for a server that can't take the clock the task asks for #632 made sync_env_clock's other refusals of a task's clock.The rest of FD-5477's scope
Already in place since #649:
sync_timeright after arming, and it waits up to 10 s for the computer's clock follower to show the gateway's current version, then refuses naming why, e.g. "it can't reach the gateway at : ". So an unreachable computer already fails the step at arm time, and arming needs nowait_followers_sof its own.sync_time, namingcomputer_settings.timezone.sync_timetakes no zone. Refused by sync_env_clock.Not needed:
sync_timedoesn't use the URL (its follower has its own), and the apps on a computer get universe time from the clock library as session programs, so none callssync_time(none of synthetic-pipelines' computer apps does).clock/v1sync_time, so a computer that follows can't be left out.Testing
sync_env_clock_test.pyclock/v1withoutset_time, a computer deployed alone, is refused before any request, saying to deploy it in a multi env (replacing the test that expected the bareEnvCapabilityUnsupported); a record that kept no card is refused saying so, with no mention of a multi env🤖 Generated with Claude Code


Confidence Score: 5/5
The PR appears safe to merge; no new actionable issues were found.Summary
sync_env_clocknow explains why an env's clock cannot be armed.Reviews (7) · Last reviewed commit: "Merge origin/main; a record that kept no..." · Reviewed by Greptile