Skip to content

chore: reconcile current main into develop - #424

Merged
Flow-Fly merged 2 commits into
developfrom
dev-423-release-reconciliation
Jul 28, 2026
Merged

chore: reconcile current main into develop#424
Flow-Fly merged 2 commits into
developfrom
dev-423-release-reconciliation

Conversation

@Flow-Fly

@Flow-Fly Flow-Fly commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Summary

Linked task issues

Test plan

  • Confirm git diff --exit-code origin/main HEAD succeeds at f053e53c5a55e73f530790de62a71d90f1d93b1a.
  • npx tsc --noEmit
  • npm run lint (0 errors; 5 existing warnings)
  • npm run test:run (112 files, 707 tests, plus shared Node ESM import)
  • npm run build
  • npm run fallow:audit (no issues in the 4 refreshed files)
  • git diff --check
  • Added-line residue, secret, binary, filename, and manifest/lock consistency checks

Known repository baseline

  • npm run format:check reports the same repo-wide 237-file formatting baseline present on current main. This reconciliation intentionally leaves it unchanged to preserve exact tree parity.

@Flow-Fly
Flow-Fly marked this pull request as ready for review July 28, 2026 12:38
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ef8b6617-adc3-4638-86fb-750b0ce22de2

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 dev-423-release-reconciliation

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

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2b8da765f0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread shared/src/project-data.ts Outdated
@Flow-Fly

Copy link
Copy Markdown
Owner Author

Reconciliation is paused after the automatic Codex exact-head review found a valid P1 legacy project compatibility regression already present in current main.

@Flow-Fly
Flow-Fly marked this pull request as ready for review July 28, 2026 13:21
@Flow-Fly

Copy link
Copy Markdown
Owner Author

Ready for owner merge

Exact reviewed head: f053e53c5a55e73f530790de62a71d90f1d93b1a

  • Complete tree is byte-for-byte identical to current main at cd62f5cf746d7dc6b93bdd56fbf9c2a334e853f4.
  • CI, server-container, and CodeRabbit status are green.
  • Automatic Codex review completed clean on the refreshed head.
  • Independent Terra exact-head review: APPROVE, no findings.
  • Former legacy empty-image P1 is fixed by merged fix(shared): preserve empty legacy image payloads #426, covered by regression tests, and its thread is resolved.
  • No unresolved review threads remain.

This PR is ready for the project owner to merge into develop. After merge, issue #423 can be reconciled/closed and delivery can proceed to #383.

@Flow-Fly
Flow-Fly merged commit 97f5ef6 into develop Jul 28, 2026
3 checks passed
@Flow-Fly
Flow-Fly deleted the dev-423-release-reconciliation branch July 28, 2026 14:53
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