Skip to content

feat(reports): leftover-map origin on leftover-map comparison leftover-pair leftover-map post leftover-map person coordinates independently of leftover-map pair leftover-map post leftover-map origin leftover-map person coordinates (v2.110.0) - #917

Draft
seonghobae wants to merge 1 commit into
feat/leftover-map-list-criterion-origin-badge-v21090from
feat/leftover-map-compare-list-post-origin-badge-v21100
Draft

feat(reports): leftover-map origin on leftover-map comparison leftover-pair leftover-map post leftover-map person coordinates independently of leftover-map pair leftover-map post leftover-map origin leftover-map person coordinates (v2.110.0)#917
seonghobae wants to merge 1 commit into
feat/leftover-map-list-criterion-origin-badge-v21090from
feat/leftover-map-compare-list-post-origin-badge-v21100

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Stacked on #916 @ 7978e925 (feat/leftover-map-list-criterion-origin-badge-v21090). leftoverMapCompareListPostBadge (ADR 0353 / v2.110.0) names leftover-map origin on leftover-map comparison leftover-pair leftover-map post leftover-map person coordinates independently of leftover-map pair leftover-map post leftover-map origin leftover-map person coordinates.

  • Distinct accessible name leftover map comparison leftover pair leftover-map post {title} at leftover-map origin ξ {person} when leftoverMapPlotCoordinatePairIsOrigin returns true (formatLeftoverMapCoordinatePair(0, 0) === (0.00, 0.00)).
  • Rank-0 unused axes still name leftover-map origin (0.00, 0.00).
  • Non-origin leftover-map comparison leftover-pair leftover-map post leftover-map person coordinates stay leftover map comparison leftover pair leftover-map post {title} at ξ {person}.
  • leftoverMapListPostBadge stays leftover-map pair leftover-map post leftover-map origin leftover-map person coordinate keys from ADR 0351.
  • leftoverMapComparePlotPostBadge stays leftover-map comparison graphic leftover-map post leftover-map origin leftover-map person coordinate keys from ADR 0349.
  • leftoverMapListCriterionBadge stays leftover-map pair leftover-map criterion leftover-map origin leftover-map item coordinate keys from ADR 0352.
  • leftoverMapCompareListCriterionBadge stays leftover-map comparison leftover-pair leftover-map criterion leftover-map item coordinate keys from ADR 0342. Do not mix feat(reports): leftover-map origin on leftover-map comparison graphic leftover-map axis ticks independently of leftover-map axis share and leftover-map singular values (v2.92.0) #877.
  • Do not invent leftover-map origin from leftover-map item coordinates ζ.
  • UI-only. No SQL. No new columns. Never invent a leftover score. Never invent a theta.

Exact head: 8192b48560a919c800ec47927adc966179a2b572

Test plan

Leave open for independent review. Do not self-approve. Do not squash-merge.


Devin Review

…r-pair leftover-map post leftover-map person coordinates independently of leftover-map pair leftover-map post leftover-map origin leftover-map person coordinates (v2.110.0)

Caption leftover-map comparison leftover-pair leftover-map post leftover-map origin leftover-map person coordinates through leftoverMapCompareListPostBadge independently of leftover-map pair leftover-map post leftover-map origin leftover-map person coordinates (ADR 0353). Distinct accessible name leftover map comparison leftover pair leftover-map post {title} at leftover-map origin ξ {person} when leftoverMapPlotCoordinatePairIsOrigin returns true. Rank-0 unused axes still name leftover-map origin (0.00, 0.00). leftoverMapListPostBadge stays leftover-map pair leftover-map post leftover-map origin leftover-map person coordinate keys. leftoverMapComparePlotPostBadge stays leftover-map comparison graphic leftover-map post leftover-map origin leftover-map person coordinate keys. leftoverMapCompareListCriterionBadge stays leftover-map comparison leftover-pair leftover-map criterion leftover-map item coordinate keys. Do not invent leftover-map origin from leftover-map item coordinates ζ. Do not mix #877.
@coderabbitai

coderabbitai Bot commented Sep 2, 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: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 759e6d3b-f84f-48f4-be67-dd9043d5757b

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

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.

Copy link
Copy Markdown
Contributor Author

Exact-head identity for independent review.

Leave open for independent APPROVE on this exact head. Do not self-approve. Do not squash-merge onto unprotected leftover bases. Issues #79 and #87 stay OPEN.

@devin-ai-integration devin-ai-integration 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.

Devin Review found 2 potential issues.

Devin Review

Comment on lines 180 to +185
if (person === null) {
return null;
}
return { key: LEFTOVER_MAP_COMPARE_LIST_POST_ACTION, values: { title, person } };
const origin = leftoverMapPlotCoordinatePairIsOrigin(person);
return {
key: origin ? LEFTOVER_MAP_COMPARE_LIST_POST_ACTION_ORIGIN : LEFTOVER_MAP_COMPARE_LIST_POST_ACTION,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 Info: Origin detection remains coordinate-local

leftoverMapCompareListPostBadge selects the origin key from validated person coordinates alone. Item coordinates cannot create or suppress this caption.

(Refers to this code)

Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Comment on lines +183 to +185
const origin = leftoverMapPlotCoordinatePairIsOrigin(person);
return {
key: origin ? LEFTOVER_MAP_COMPARE_LIST_POST_ACTION_ORIGIN : LEFTOVER_MAP_COMPARE_LIST_POST_ACTION,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝 Info: Displayed precision defines origin

Tiny finite coordinates that format as (0.00, 0.00) receive the origin label. Existing helpers and ADR 0353 use this displayed-coordinate contract.

Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-09-02T21:16 KST write retry. Exact head 8192b48560a919c800ec47927adc966179a2b572 has Devin COMMENT only — not independent APPROVE. No self-approval. Leave stacked leftover-map PRs open. Next unique increment after ADR 0353 / v2.110.0 is leftover-map origin on leftover-map comparison leftover-pair leftover-map criterion leftover-map item coordinates as leftoverMapCompareListCriterionBadge (ADR 0354 / v2.111.0), independently of leftover-map comparison leftover-pair leftover-map post leftover-map origin leftover-map person coordinates. Do not mix #877. Issues #79 and #87 stay OPEN.

Copy link
Copy Markdown
Contributor Author

Stacked next buyer increment as #918 (feat/leftover-map-compare-list-criterion-origin-badge-v21110 @ a91d2ec56bad922f9159e17649f77d229a0f98d3) onto this exact head 8192b485. leftoverMapCompareListCriterionBadge (ADR 0354 / v2.111.0) names leftover-map origin on leftover-map comparison leftover-pair leftover-map criterion leftover-map item coordinates independently of leftover-map comparison leftover-pair leftover-map post leftover-map origin leftover-map person coordinates. Distinct accessible name leftover map comparison leftover pair leftover-map criterion {label} at leftover-map origin ζ {item}. Rank-0 unused axes still name leftover-map origin (0.00, 0.00). leftoverMapCompareListPostBadge stays leftover-map comparison leftover-pair leftover-map post leftover-map origin leftover-map person coordinate keys this increment. Do not mix #877. Do not mix #890 leftoverMapPlotOriginBadge reconstruction into #918. Leave #917 and #918 open for independent review. Do not self-approve. Do not squash-merge. Issues #79 and #87 stay OPEN.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant