Skip to content

docs(development): move the streaming construction plan out of the repo into issue #1478 - #1479

Merged
DecisionNerd merged 1 commit into
mainfrom
docs/streaming-construction-plan-to-issue
Sep 19, 2026
Merged

DecisionNerd merged 1 commit into
mainfrom
docs/streaming-construction-plan-to-issue

Conversation

@DecisionNerd

@DecisionNerd DecisionNerd commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

Why

The streaming construction plan — the plan of record for #1387 (the ingest throughput floor) and #1456 (the construction foundation) — went through eleven revisions in two days. As a repository file every revision needed a pull request. The maintainer's instruction: "we should move it out of the repo and move it into the issues or milestone body so we are not dealing with PRs."

What

Nothing else references the file (checked across *.md, *.yml, *.toml, *.py).

Where the plan lives now

#1478 — plan: streaming construction plan (rev 12) — the #1387 floor, per resource, in milestone M5. It is the plan of record and the closing issue for the ingest-throughput workstream (#1387 sits under it as a sub-issue; #1456 sits under #1387). It is not M5's closing tracker; that remains #735.

Rev 12 is not a copy of rev 11. Its section 0 records every claim rev 11 made that the F1 device measurement, the perf stat / N-way scaling measurement and the external review refuted, and what replaced each. Rev 11's text stays in history at 7b0113dd.

The status headers on #1387 and #1456, the M5 description, and the local pointer files are repointed to #1478 alongside this PR.

Gates

  • scripts/ci/adr-index.py check — ok, 35 active and 3 superseded records agree
  • scripts/source_size_policy.py — 693 files checked, 0 violations
  • scripts/ci/test-text-checkout-policy.py — 3,603 tracked paths resolve to LF
  • make pre-push-fast — passed

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

…po into issue #1478 (rev 12)

The plan of record for #1387 / #1456 is a living document that was revised
eleven times in two days. Holding it as a repository file meant a pull
request per revision. The maintainer ruled that it moves into the tracker
instead: rev 12 is issue #1478 (milestone M5), and a revision is now an
edit to that body plus a comment saying what changed.

This removes docs/development/streaming-construction-plan.md (rev 11, PR
#1475) and its row in docs/engineering/README.md. Rev 11's content is
superseded by #1478 section 0, which records each withdrawn claim and what
replaced it; the file stays in history at 7b0113d.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: CurateLabs/graphforge/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 21be8097-2551-4bdf-bbd1-bfb9b99904ec

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added documentation Improvements or additions to documentation release:none No release note or version impact labels Sep 19, 2026
@DecisionNerd
DecisionNerd added this pull request to the merge queue Sep 19, 2026
Merged via the queue into main with commit 7645f96 Sep 19, 2026
23 checks passed
@DecisionNerd
DecisionNerd deleted the docs/streaming-construction-plan-to-issue branch September 19, 2026 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation release:none No release note or version impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant