🐛 app: fix proposal block override time

[dieguezguille]

Summary by CodeRabbit

• Bug Fixes
  • Fixed proposal block override timestamp calculation to ensure accurate block timing during proposal simulations and delayed executions.

[coderabbitai]

Walkthrough

This PR introduces a patch-level update for the @exactly/mobile package that fixes the proposal block override time. The changeset metadata is added, and the simulation utility is updated to add an extra 1 to block timestamp calculations for both initial and delayed execution blocks.

Changes

Cohort / File(s)	Summary
Changeset Metadata .changeset/wild-geckos-remain.md	New changeset file declaring a patch-level update for @exactly/mobile with a note about the proposal block override time fix.
Simulation Utility src/utils/useSimulateProposal.ts	Updated block timestamp calculations in useSimulateBlocks by adding + 1n to both the initial block override and delayed execution block override computations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~7 minutes

Possibly related PRs

• 🐛 app: align propose and execute block timestamps #914: Modifies the same src/utils/useSimulateProposal.ts file to adjust how propose and execute block timestamps are computed and aligned.

Suggested labels

bug

Suggested reviewers

• franm91
• cruzdanilo

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix proposal block override time' directly corresponds to the main change: a fix to block timestamp calculations in useSimulateProposal affecting proposal block override behavior.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch simulate

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 96713f67-6504-4c0c-8400-7589b8ed726a

📥 Commits

Reviewing files that changed from the base of the PR and between a43a2e5 and 45fcb30.

📒 Files selected for processing (2)

• .changeset/wild-geckos-remain.md
• src/utils/useSimulateProposal.ts

[coderabbitai]

🧹 Nitpick | 🔵 Trivial

Add a regression test for timestamp boundary behavior.

Given this is a precision timing fix, add a focused test for the equality boundary (timestamp + delay) so this doesn’t regress.

Also applies to: 184-187

[gemini-code-assist]

Code Review

This pull request updates the useSimulateProposal utility to increment the block override timestamp by 1n in two simulation scenarios. This adjustment ensures that the simulated block time is strictly greater than the current timestamp, preventing potential simulation errors. A changeset file has also been included to document this patch. I have no feedback to provide.

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

[sentry]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
662	1	661	0

View the full list of 1 ❄️ flaky test(s)

local::local

Flake rate in main: 100.00% (Passed 0 times, Failed 1 times)

Stack Traces | 350s run time

Element not found: Text matching regex: Spending limit

To view more test analytics, go to the Prevent Tests Dashboard