Skip to content

Clarify auto-backgrounding on servers inside of agents. - #14573

Open
matthewp wants to merge 5 commits into
mainfrom
docs/background-windows
Open

Clarify auto-backgrounding on servers inside of agents.#14573
matthewp wants to merge 5 commits into
mainfrom
docs/background-windows

Conversation

@matthewp

@matthewp matthewp commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Description (required)

Backgrounding on Windows does not work because the agent kills the process anyways. So don't attempt to background and allow the agent to manager the server itself.

References

For withastro/astro#18029

@matthewp
docs: clarify auto-background does not happen on Windows
71bbcff 
When an AI agent is detected, astro dev and astro preview auto-background
on macOS and Linux only. On Windows the server stays in the foreground so
the agent can manage the process directly; --background remains the
explicit opt-in. See withastro/astro#18029.
@matthewp
docs: drop Windows Job Objects implementation detail from background …
d356158 
…mode
@matthewp matthewp mentioned this pull request Sep 17, 2026
Merged
@astrobot-houston

astrobot-houston commented Sep 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the tracking.ignoredKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
guides/build-with-ai.mdx Source changed, localizations will be marked as outdated.
reference/cli-reference.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@github-actions

github-actions Bot commented Sep 17, 2026 •
edited
Loading

Copy link
Copy Markdown

Preview deployment

✅ Deployment complete!

ArmandPhilippot
ArmandPhilippot approved these changes Sep 17, 2026
View reviewed changes

@ArmandPhilippot ArmandPhilippot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks Matthew, I left some suggestions for the Build with AI guide. Mostly nitpicks, so approving anyway.

Comment thread src/content/docs/en/guides/build-with-ai.mdx Outdated
Comment thread src/content/docs/en/guides/build-with-ai.mdx Outdated
Comment thread src/content/docs/en/guides/build-with-ai.mdx
Comment on lines 371 to 372
If you are not using an AI coding agent, the server starts in the foreground process and logs to the terminal.

@ArmandPhilippot ArmandPhilippot Sep 17, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Only if you apply my last suggestion:

Suggested change

@ArmandPhilippot ArmandPhilippot Sep 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a reminder you forgot this suggestion.

I know the diff looks weird because this line haven't been updated... But we now say that on l367, so we don't want to duplicate the info.

After that, this PR is ready to go with the next release!

@ArmandPhilippot

ArmandPhilippot commented Sep 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

And I opened #14574 to fix the deploy preview, following the error message suggestion: https://github.com/withastro/docs/actions/runs/35225652603/job/105216447524

Edit: fix merged! If you update your branch this should work now!

matthewp and others added 3 commits September 17, 2026 10:49
@matthewp @ArmandPhilippot
Update src/content/docs/en/guides/build-with-ai.mdx
b62b74f 
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
@matthewp @ArmandPhilippot
Update src/content/docs/en/guides/build-with-ai.mdx
e564aea 
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
@matthewp
Merge branch 'main' into docs/background-windows
ff8d9fa
@ArmandPhilippot ArmandPhilippot added improve or update documentation Enhance / update existing documentation (e.g. add example, improve description, update for changes) merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) labels Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ArmandPhilippot ArmandPhilippot ArmandPhilippot approved these changes

Assignees

No one assigned

Labels

improve or update documentation Enhance / update existing documentation (e.g. add example, improve description, update for changes) merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@matthewp @astrobot-houston @ArmandPhilippot