Skip to content

Conversation

@ematipico
Copy link
Member

Changes

This PR handles the invalidation of the manifest module when changes occur in the srdDir directory.

When a path that changed is detected, we invalidate the module itself, so the new routes can be calculated and passed to the DevApp

Testing

Manual testing by changing/deleting/adding routes inside the pages/ folder, and made sure that the data is up-to-date.

Docs

N/A

@changeset-bot
Copy link

changeset-bot bot commented Oct 14, 2025

⚠️ No Changeset found

Latest commit: b44598c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Oct 14, 2025
};
});
let environment = server.environments.ssr;
if (isRunnableDevEnvironment(environment)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

What happens in other enviornments?

Copy link
Member Author

Choose a reason for hiding this comment

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

I removed the check, it seems to be working even without it

@ematipico ematipico requested a review from ascorbic October 14, 2025 13:30
Copy link
Contributor

@matthewp matthewp left a comment

Choose a reason for hiding this comment

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

Lgtm

@ematipico ematipico merged commit 9d2b501 into feat/environment-api Oct 14, 2025
7 of 19 checks passed
@ematipico ematipico deleted the env-api/hmr-manifest branch October 14, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants