Skip to content

An answer can't bind to an artifact page that changed since it was asked - #120

Merged
hhff merged 1 commit into
mainfrom
artifact-answer-binds
Nov 6, 2026
Merged

hhff merged 1 commit into
mainfrom
artifact-answer-binds

Conversation

@hhff

@hhff hhff commented Nov 6, 2026

Copy link
Copy Markdown
Member

What

  • devteam.mjs: artifactLinksMoved(v) returns why an answer can't bind when a question's pinned artifact link no longer matches its fingerprint, or is no longer available. Both answer and ownerAnswer (every use of an answer as an approval) refuse with it.
  • Tests: in the artifact-links test, answering after the draft changed is refused, and so is answering after the draft is gone.

Why

This was blocker 2 of the #118 review. A question pins an artifact link by its fingerprint, but the owner could answer, or the supervisor could use that answer, after the page had changed, so the "yes" covered a page the owner never saw. The dashboard only showed a badge. It's the same class of hole as #116.

Migration

None. Questions whose linked artifact changed must be asked again.

🤖 Generated with Claude Code

A question can link an artifact pinned by fingerprint when it was asked. If the
page changed afterwards (or its draft is gone), the owner would be answering,
or an answer would be approving, a page they never saw: the dashboard only
showed a badge. answer now refuses while a linked artifact differs from its
pin or is no longer available, and so does every use of an answer as an
approval (ownerAnswer): the supervisor asks again about the page as it is.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@hhff

hhff commented Nov 6, 2026

Copy link
Copy Markdown
Member Author

Merging for v0.20.2 (upstream loop). Main + #119 + #120 passed the full suite locally on Node 24.14.1 (129/129). CI 18/20/22/24 is green at this head, with current main merged in.

@hhff
hhff merged commit 5b2b5f8 into main Nov 6, 2026
4 checks passed
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