Skip to content

docs(desktop): update Status Tracker rows post-dry-run#649

Merged
jeevanpillay merged 2 commits intomainfrom
docs/codex-gap-status-tracker-callout
May 6, 2026
Merged

docs(desktop): update Status Tracker rows post-dry-run#649
jeevanpillay merged 2 commits intomainfrom
docs/codex-gap-status-tracker-callout

Conversation

@jeevanpillay
Copy link
Copy Markdown
Member

@jeevanpillay jeevanpillay commented May 6, 2026

Summary

Flip the three IN PROGRESS rows in the original 2026-04-23 Status Tracker (§2 source-map upload, §6 multi-arch matrix, §13 build-metadata stamping) to DONE, with inline notes citing the closing PRs (#621 for the original landings; #639/#641/#642 for dry-run-era refinements). Git history preserves the 2026-04-23 snapshot.

Also trims the Legend — IN PROGRESS no longer needed since no rows use it. Brief parenthetical notes the historical convention.

Why this approach (not a callout pointing elsewhere)

First commit on this branch added a blockquote callout above the table pointing at the post-dry-run section. On reflection, fragmented documentation: the table should reflect current truth. Audit trail is preserved by git history and by the existing Status Update sections at the bottom (which already enumerate "what changed when").

Test plan

  • No `IN PROGRESS` markers remain in the Status Tracker table
  • Remaining `IN PROGRESS` mentions in the doc are in the post-2026-04-24 Status Update sections (correct audit-trail context)
  • Markdown renders cleanly

🤖 Generated with Claude Code

Three IN PROGRESS rows in the original 2026-04-23 Status Tracker (§2
source-map upload, §6 multi-arch matrix, §13 build-metadata stamping)
all landed via PR #621 (2026-04-24), but a top-down reader doesn't
discover that until they scroll past 470 lines to the Status Update
sections. Add a one-line callout right above the table pointing at
the post-dry-run section and the dry-run report.

Audit trail preserved — table itself unchanged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lightfast-app Ready Ready Preview, Comment May 6, 2026 11:26am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
lightfast-platform Ignored Ignored Preview May 6, 2026 11:26am
lightfast-www Ignored Ignored Preview May 6, 2026 11:26am

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 6, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • thoughts/shared/research/2026-04-23-codex-vs-lightfast-desktop-production-gap.md is excluded by !thoughts/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d326f1d6-9ae5-4844-870e-a23dcdd6715c

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/codex-gap-status-tracker-callout
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch docs/codex-gap-status-tracker-callout

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

Per follow-up feedback: actually flip the three IN PROGRESS rows to
DONE rather than adding a callout that points readers elsewhere. Git
history preserves the 2026-04-23 snapshot; the inline notes now cite
the closing PRs (#621 for the original landings; #639/#641/#642 for
the dry-run-era refinements to source-map upload + release stamping).

Drops IN PROGRESS from the Legend with a brief parenthetical noting
the historical convention; replaces the callout added in the prior
commit with these in-place updates.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jeevanpillay jeevanpillay changed the title docs(desktop): add Status Tracker post-dry-run callout docs(desktop): update Status Tracker rows post-dry-run May 6, 2026
@jeevanpillay jeevanpillay merged commit 28eae95 into main May 6, 2026
14 checks passed
@jeevanpillay jeevanpillay deleted the docs/codex-gap-status-tracker-callout branch May 6, 2026 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant