docs: extract ADRs and knowledge from 6 completed specs - #10103
Open
polmichel wants to merge 7 commits into
Open
docs: extract ADRs and knowledge from 6 completed specs#10103polmichel wants to merge 7 commits into
polmichel wants to merge 7 commits into
Conversation
Extract ADR 0011 from ifc-2273-local-computation-jinja2 and archive the spec. The feature's mechanics were already documented in computed-attributes.md by the implementation; only the architectural decision (inline evaluation for local changes, background tasks for remote) was missing as a record. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Extract ADR 0012 from ifc-2704-incremental-merge-regen and archive the spec. Add a best-effort-side-effect broad-catch rule to the Python exception-handling guideline, generalized from the merge-diff capture. The runtime flow was already documented in selective-merge-regeneration.md; update its two spec references to the archive path. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Extract ADRs 0013-0015 from ifc-2755-webhook-delivery-operability and archive the spec: deliveries as Prefect-run objects, generic per-task recovery actions with polymorphic task typing, and uniform bounded retry. Document the polymorphic task typing in async-tasks.md, and correct the webhooks.md retry paragraph, which described a transient flag no longer present in the code, to the uniform retry that landed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Custom webhook headers are already documented in webhooks.md by the landed implementation; this early draft has no research.md and yields no new content. Record and archive it, and fix the webhooks.md See Also link to the archived path (and its inaccurate label). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…mented) The inline local computed-attribute behavior is already documented in computed-attributes.md, and the decision is recorded in ADR 0011 (from the sibling ifc-2273 spec). This early draft has no research.md and yields no new content; record and archive it. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Extract the reusable read_only and display=EXTRA schema-flag semantics into schema-definitions.md and archive infp-556-auto-create-groups. The origin attribute decision is already recorded in ADR 0005, and the SSO pipeline in authentication.md; update their spec references to the archive path. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
github-actions
Bot
added
the
type/spec
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
No issues found across 81 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. Archives completed specs and extracts decisions into ADRs/knowledge. Purely documentation change with no code, configuration, or operational impact.
Re-trigger cubic
Add the five ADRs extracted from the archived specs to the Current ADRs list. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
polmichel
force-pushed
the
pmi-20260731-speckit-extract
branch
from
July 31, 2026 13:41
714778d to
ad33a4e
Compare
polmichel
changed the title
polmichel
marked this pull request as ready for review
July 31, 2026 13:52
ajtmccarty
reviewed
Jul 31, 2026
|
|
||
| ### Best-effort side effects degrade to a safe fallback | ||
|
|
||
| <!-- Extracted from specs/ifc-2704-incremental-merge-regen on 2026-07-31 --> |
Contributor
There was a problem hiding this comment.
I don't know if these are valuable
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
2 participants
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.
What
Runs
speckit.opsmill.extractover 6 completed spec directories, promoting durable decisions and reference material intodev/adr/,dev/knowledge/, anddev/guidelines/, then archiving each spec underdev/specs/archive/. One commit per spec, plus an index update.Most feature mechanics were already documented by the implementations; extraction filled the remaining gaps (chiefly the decision records) and moved the specs out of the active area.
Added ADRs
Knowledge / guideline changes
knowledge/backend/async-tasks.md— new "Task typing (polymorphic)" section (TaskNodeInterface,TASK_TYPES,resolve_type).knowledge/backend/webhooks.md— corrected the retry paragraph, which described atransientclassifier flag no longer present in the code, to the uniform bounded retry that landed.knowledge/backend/schema-definitions.md— new "Read-only and Display Tier" section (read_only,display=SchemaAttributeDisplay.EXTRA).guidelines/backend/python.md— new "Best-effort side effects degrade to a safe fallback" rule under Exception Handling.Archived (no new content — already documented)
infp-445-webhook-headersandinfp-460-local-computed-attributeshave noresearch.md; their behavior already lives inwebhooks.md/computed-attributes.md. Recorded and archived.infp-556-auto-create-groups— the decision is already ADR 0005; only the reusable schema-flag knowledge was new.All 6 specs moved to
dev/specs/archive/, each with anEXTRACTED.mdandStatus: Extracted; the inbound doc links were repointed to the archive path.Notes
ifc-2437-merge-failure-recoverywas intentionally left out of this PR and remains an active spec..specify/extension reinstall (opsmill-speckitv1.1.0) is not part of this branch; it stayed in the working tree as an unrelated tooling change.🤖 Generated with Claude Code