Skip to content

Commit a405e90

Browse files
UN-3479 [MISC] Make clone code comments concise and generic
Drop persona-mode markers and conversational phrasing from comments. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CsGrHbs5SWmQkKqiimg6CF
1 parent bd05099 commit a405e90

3 files changed

Lines changed: 13 additions & 16 deletions

File tree

src/unstract/clone/context.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -152,8 +152,8 @@ class CloneContext:
152152
def feature_present(self, client: "PlatformClient", path: str) -> bool:
153153
"""Is ``path`` (a feature's list endpoint) installed on ``client``?
154154
155-
Memoised per run. # ponytail: plain dict, no lock — probing runs in
156-
the single-threaded orchestrator loop, before any parallel_map fan-out.
155+
Memoised per run. Plain dict, no lock — probing runs in the
156+
single-threaded orchestrator loop, before any parallel_map fan-out.
157157
"""
158158
key = (id(client), path)
159159
cached = self.probe_cache.get(key)

src/unstract/clone/phases/agentic_studio.py

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,8 @@
2828
registry are left unexported.
2929
3030
Org-level ``AgenticSetting`` rows (global key/value, no project FK) are cloned
31-
once as a flat adopt-by-key / create pass.
32-
33-
# ponytail: settings are org-global singletons keyed by a globally-unique
34-
# ``key`` — not the per-project config the brief described (that config lives
35-
# on the project itself). They're cloned flatly here, not per project.
31+
once as a flat adopt-by-key / create pass — they are org singletons, not
32+
per-project config.
3633
"""
3734

3835
from __future__ import annotations
@@ -222,10 +219,10 @@ def _replicate_share(
222219
# The project list/detail share the same serializer, so the source row
223220
# already carries shared_users (target-mappable user pks), shared_to_org
224221
# and created_by — no detail fetch needed.
225-
# ponytail: agentic group sharing is deferred — shared_groups is
226-
# polymorphic/read-only on the project serializer (share via the detail
227-
# PATCH only reaches shared_users + shared_to_org), so include_groups is
228-
# off and a source group share yields a single warning.
222+
# Group sharing isn't replicated: shared_groups is polymorphic/read-only
223+
# on the project serializer (the detail PATCH only reaches shared_users +
224+
# shared_to_org), so include_groups is off and a source group share
225+
# yields a single warning.
229226
replicate_share(
230227
self.ctx,
231228
apply_fn=lambda p: self.ctx.target.update_agentic_project_share(
@@ -414,7 +411,7 @@ def _republish_registry(
414411
return
415412

416413
if not src_regs:
417-
# ponytail: nothing to republish — project was never exported.
414+
# Nothing to republish — project was never exported.
418415
return
419416

420417
try:

src/unstract/clone/phases/lookups.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -428,10 +428,10 @@ def _replay_one_version(
428428
name, src_lookup_id, tgt_lookup_id, src_version_id, detail, result, lock
429429
)
430430

431-
# ponytail: assignment_values_snapshot is derived by the backend at
432-
# publish time from then-existing assignments; we publish before
433-
# assignments are recreated, so historical snapshots may differ from
434-
# source. Structure + pinning reproduce; frozen values are best-effort.
431+
# assignment_values_snapshot is derived by the backend at publish time
432+
# from then-existing assignments; publishing happens before assignments
433+
# are recreated, so historical snapshots may differ from source.
434+
# Structure + pinning reproduce; frozen values are best-effort.
435435
try:
436436
published = self.ctx.target.publish_lookup_version(
437437
tgt_lookup_id,

0 commit comments

Comments
 (0)