starred/windmill
1
0
Fork 0
mirror of https://github.com/windmill-labs/windmill.git synced 2026-10-13 08:02:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

docs: state the value precedence the reader would otherwise get backwards

Browse Source 
Local review nits.

`readGlobalDraftValue`'s doc still described the order the previous commit
changed — it read as cell then backend, with the resolved value as a footnote,
while the code is cell, then resolved value, then backend. A future edit
following the doc would reintroduce the bug the reorder fixed.

The `probeUnknown` refusal now says why it stops a read, unlike the listing
failure ten lines above which hands reads the addressed path: that branch knows
nothing is staged under the name, this one knows something is, so the name is
ambiguous rather than merely unresolved. Two rounds have now read the asymmetry
as an oversight; it is the stop that keeps the destructive tools, which reach it
as reads, off a namesake draft.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
Guilhem Lemouel
2026-09-29 03:43:09 -07:00
co-authored by Claude Opus 5
parent a055421ba4
commit a5f307fd24
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend/src/lib/components/copilot/chat/global/userDraftAdapter.ts
+10 -3
View File
@@ -432,6 +432,12 @@ async function storagePathForChosenName(
// Something is staged under this name, and whether the path is also a deployed item's
// own is what says which draft the call means. Answering it either way could delete or
// deploy an unrelated draft.
//
// Reads stop here too, unlike the listing failure above, which hands them the addressed
// path. That branch knows nothing is staged under the name; this one knows something is,
// so the name is genuinely ambiguous and saying so beats reading whichever item the path
// happens to hold. The destructive tools reach this as reads — they declare no
// `forWrite` — and this is the stop that keeps them off the namesake.
throw new Error(
`Could not tell whether something is already deployed at "${path}", so the draft staged ` +
`under that name cannot be identified. Try again.`
@@ -552,9 +558,10 @@ async function fetchBackendDraftValue(
return resp.value ?? undefined
}
/** Draft VALUE at a resolved target: cell-if-present (the user's freshest in-tab edits)
* else the current user's backend draft. Falls back to what resolving the target already
* read, unless `fresh` — for a caller that has since changed the draft it is reading. */
/** Draft VALUE at a resolved target, in order: the cell if one is open (the user's freshest
* in-tab edits), else the value resolving the target already read, else the current user's
* backend draft. `fresh` skips the resolved value — for a caller that has since changed the
* draft it is reading. */
export async function readGlobalDraftValue<V>(
workspace: string,
type: WorkspaceItemType,