Narrow two discussions to the half that is still open - #175
Merged
Conversation
An open-issue review found both discussions asking for two quantities each, one of which #65 has since sourced. Left as they were, they would send a curator after finished work. ring_dimensions_unsourced "FtsZ filament length and Z-ring thickness" filament length 40-160 nm and diameter ~5 nm now carry verbatim snippets. The Z-ring's radial thickness does not: the review reports it only negatively -- it did not increase even under FtsZ overexpression -- and gives no figure. The ~13 nm THICKNESS on the record is the filament-to-membrane spacing, a different quantity, as its notes say. needle_dimensions_unsourced "needle length and channel diameter" channel diameter ~2.5 nm, filament width ~6 nm and base diameter ~25 nm now carry verbatim snippets. The length does not, because the source says only that the needle "protrudes several nanometers" and gives no figure. It was deliberately not estimated from that phrasing. Both prompts now name only what is missing, and both rationales say why the other half is closed so the narrowing is auditable rather than looking like the question shrank on its own. just qc passes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SiYu4XXes2B6dU926yk4wm
This was referenced Sep 3, 2026
Self-review of the previous commit found it had replaced each discussion's rationale rather than adding to it, and the deleted text was the more valuable half: A first pass added filament length and ring thickness attributed to McQuillen & Xiao 2020 without reading it; both were removed (#91). A first pass added ~45-80 nm needle length and a ~2-3 nm channel attributed to Deng et al. 2017 on the strength of its title alone. That is the record of specific numbers fabricated from paper titles and the #91 removal that followed -- the institutional memory that stops the mistake recurring -- and it was overwritten with "Narrowed after #65". The schema says rationale is "why this matters", which the original answered and the replacement did not. Both rationales are restored byte-identical to main. The narrowing moves to `notes`, which the schema provides and neither discussion was using (#179). posed_date is restored to 2026-08-31 (#180). It records when a question was raised; narrowing does not re-raise it, and moving it forward made a question open since 31 August look two days old -- backwards for a field whose only use is showing how long something has sat there. The value written was not even the edit's own date. docs/CURATION.md now states how a discussion's fields age: prompt may be narrowed, rationale is appended to and never replaced, notes holds current state, posed_date never moves, and issue links belong in `evidence` (#181, left open as a sweep). just qc passes. Closes #179 Closes #180 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SiYu4XXes2B6dU926yk4wm
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.
From an open-issue review. Not an issue fix — a finding the review turned up
outside the queue.
Both discussions asked for two quantities, and #65 has since sourced one of
each. Left as they were, a curator reading them would go looking for work that
is already done.
divisome_complex#ring_dimensions_unsourcedtype_iii...#needle_dimensions_unsourcedNeither remaining half is an oversight:
not increase even under FtsZ overexpression — and gives no figure. The ~13 nm
THICKNESSon the record is the filament-to-membrane spacing, a differentquantity, and its evidence notes already say so.
bacterial surface". That was deliberately not turned into a number.
Both rationales now record why the other half closed, so the narrowing is
auditable rather than looking like the question quietly shrank.
just qcpasses.🤖 Generated with Claude Code
https://claude.ai/code/session_01SiYu4XXes2B6dU926yk4wm