fix: Set stale status to false on cell transition to idle

[wasimsandhu]

📝 Summary

Fixes regression introduced by #1625.

🔍 Description of Changes

📋 Checklist

• [x] I have read the contributor guidelines.
• [x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
• [ ] I have added tests for the changes made.
• [x] I have run the code and verified that it works as expected.

📜 Reviewers

@akshayka OR @mscolnick

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
marimo-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 17, 2024 5:26am
marimo-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 17, 2024 5:26am

[github-actions[bot]]

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.6.20-dev5

[akshayka]

@wasimsandhu one more issue:

I ran marimo tutorial dataflow. The following cells are stale on main but they shouldn't be stale:

[mscolnick]

@wasimsandhu - i think you may want to put unintialized later in the if/else statement, to take a later priority

[akshayka]

I think the issue is actually that lastExecutionTime is empty for errored cells -- cells with eg MultipleDefinitionErrors don't actually run. (cc @wasimsandhu)

Minimal repro:

import marimo

__generated_with = "0.6.19"
app = marimo.App()

@app.cell
def __():
    x = 0
    x
    return x,

@app.cell
def __():
    x = 1
    x
    return x,

if __name__ == "__main__":
    app.run()