Sequence #88 ahead of #171 - #204
Merged
Merged
Conversation
#171 requires batch results to retain path/parent information so duplicate names stay distinguishable, and its proposed input requests a path field. TagItem already exposes parentID, parentName, and path. ProjectItem exposes no folder or path field at all -- that is what #88 adds. Shipping #171 before #88 would mean returning indistinguishable entries for same-named projects, which is the failure its acceptance criteria exist to prevent, and a partial contract would be hard to tighten afterwards. #88 therefore moves to the front of the delivery order and #171 follows, shipping its contract in full for both tools. #87 keeps its slot and is noted as the intended home for stalled/health questions, so batch resolution is not widened to answer them. Also records the open includeTaskCounts question on #171 with its measured cost: roughly +1.3s and about double the response bytes on a 392-project database. Validation impact: docs. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
1 participant
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.
Docs-only. Impact:
docs(focusrelay-dev validate --impact docspasses).#171 requires batch results to retain path/parent information so duplicate names stay distinguishable, and its proposed input requests a
pathfield.TagItemalready exposesparentID,parentName, andpath.ProjectItemexposes no folder or path field at all — that is exactly what #88 adds.Shipping #171 first would return indistinguishable entries for same-named projects, which is the failure its acceptance criteria exist to prevent, and a partial contract is hard to tighten afterwards. So #88 moves to the front and #171 follows with its contract intact for both tools.
#87 keeps its slot, now noted as the intended home for stalled/health questions so batch resolution is not widened to answer them.
Also records the open
includeTaskCountsquestion on #171 alongside its measured cost — roughly +1.3s and about double the response bytes on a 392-project database — so whoever picks it up starts from evidence rather than re-deriving it.🤖 Generated with Claude Code