Skip to content

feat: dynamic gallery index refresh on marimo run <folder> --watch#8460

Merged
mscolnick merged 3 commits intomarimo-team:mainfrom
peter-gy:ptr/refresh-gallery-in-watch-mode
Feb 25, 2026
Merged

feat: dynamic gallery index refresh on marimo run <folder> --watch#8460
mscolnick merged 3 commits intomarimo-team:mainfrom
peter-gy:ptr/refresh-gallery-in-watch-mode

Conversation

@peter-gy
Copy link
Contributor

This PR is complementary to #8459 based on request outlined in #8444. It supports dynamic inclusion of notebooks into gallery index page spawned by marimo run <folder> --watch.

Opening this only as a draft for the sake of discussion (cc @mscolnick). This feature comes with the security consideration that a notebook allowing file upload / folder mutation could add a new (potentially malicious) .py file to the served gallery folder which executes code on open.

@peter-gy peter-gy added the enhancement New feature or request label Feb 25, 2026
@vercel
Copy link

vercel bot commented Feb 25, 2026

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

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Feb 25, 2026 4:34pm

Request Review

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Feb 25, 2026
folder index on each workspace request. This means added and removed notebooks
appear after refreshing or revisiting the gallery page.

!!! warn "Security note for watched folders"
Copy link
Contributor

Choose a reason for hiding this comment

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

i think this is sufficient, since already technically a user could create a file named after a common built-in (e.g called os.py) which would get imported when a notebook runs import os. so users should generally be cautious of allowing file uploads and saving to disk

mscolnick
mscolnick previously approved these changes Feb 25, 2026
@peter-gy peter-gy marked this pull request as ready for review February 25, 2026 14:56
@peter-gy peter-gy requested a review from akshayka as a code owner February 25, 2026 14:56
@peter-gy
Copy link
Contributor Author

@mscolnick just resolved the merge conflicts

@mscolnick mscolnick merged commit a3bcf22 into marimo-team:main Feb 25, 2026
48 of 87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants