frontend-patterns/research/reviews/ZulipTargetLookupSources.json
tegwick fad8abdc2c Record LEARN-0017: traced Zulip target-lookup/anchor guard gap (corroboration)
Second cycle in comparison group B-independent-navigation-source-v1, a
naturally-motivated follow-up to LEARN-0014's ZN07 finding rather than a
manufactured repetition. Traces the target-message lookup and anchor-
navigation fetch continuation to their unconditional completion points in
message_fetch.ts: neither participates in the opt-in current-list-identity
guard used elsewhere in the same file, and no located test covers a
stale-navigation scenario. No reproduction or defect claim; static source
read only. CAND-0009/CAND-0010 evidence and gaps updated accordingly.

Declined to reopen the paused local navigation fixture to force a third
cycle in an existing group, per the prior agreed pause and WP-0006 T03's
own rule against manufacturing cycles to hit a count.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Assistant: claude-code
Assistant-Model: sonnet
Assistant-Process: 290719@bnt-lap001
Assistant-Session: 55cbe504-1bfe-4932-b761-29016523c87b
2026-09-27 23:21:02 +02:00

50 lines
2.6 KiB
JSON

{
"repository": "zulip/zulip",
"revision": "5b06fa3d34f28fa01f459fbf3ba053e3c0858e9c",
"accessed": "2026-09-27",
"method": "Static selected-range inspection only; no application, database or upstream test executed",
"preregistration": "884803c",
"files": [
{
"path": "web/src/message_view.ts",
"sha256": "8dae489715846f1ded057663c89e1852ae871b7e3a1d7d7d04773c18429a4d89",
"lines": 1801,
"raw_url": "https://raw.githubusercontent.com/zulip/zulip/5b06fa3d34f28fa01f459fbf3ba053e3c0858e9c/web/src/message_view.ts",
"reviewed_ranges": [
{"start": 640, "end": 700, "note": "auxiliary target-message lookup (channel.get to /json/messages/:id, success/error re-entering show)"},
{"start": 998, "end": 1074, "note": "navigate_to_anchor_message and its fetch continuation"}
]
},
{
"path": "web/src/message_fetch.ts",
"sha256": "70b4367e41313789850026398727c8132ce3347229dabca226042b1d55f2693e",
"lines": 857,
"raw_url": "https://raw.githubusercontent.com/zulip/zulip/5b06fa3d34f28fa01f459fbf3ba053e3c0858e9c/web/src/message_fetch.ts",
"reviewed_ranges": [
{"start": 114, "end": 128, "note": "load_messages_around_anchor"},
{"start": 157, "end": 213, "note": "process_result, unconditional opts.cont(data, opts) call"},
{"start": 213, "end": 280, "note": "get_messages_success"},
{"start": 418, "end": 510, "note": "load_messages including the opts.msg_list !== message_lists.current error-path guard"}
]
},
{
"path": "web/tests/message_fetch.test.cjs",
"sha256": "affadd8643b3adc9ac82baf44b820cb88fec4d1feb627a9ff169053453340b11",
"lines": 40,
"raw_url": "https://raw.githubusercontent.com/zulip/zulip/5b06fa3d34f28fa01f459fbf3ba053e3c0858e9c/web/tests/message_fetch.test.cjs",
"reviewed_ranges": [
{"start": 1, "end": 40, "note": "whole file (already reviewed at ZN08); confirms only date-anchor parameter parsing is tested, not the cont/current-list race"}
]
},
{
"path": "web/tests/message_view.test.cjs",
"sha256": "5f9eb419db519a9a1493f48ae795ed4feee7aad6594d5497af53b776451af64e",
"lines": 1308,
"raw_url": "https://raw.githubusercontent.com/zulip/zulip/5b06fa3d34f28fa01f459fbf3ba053e3c0858e9c/web/tests/message_view.test.cjs",
"reviewed_ranges": [
{"start": 1, "end": 40, "note": "load_messages_around_anchor mock setup"},
{"start": 1106, "end": 1244, "note": "fast_track_current_msg_list_to_anchor tests; no target_id/fetched_target_message match found in file"}
]
}
]
}