Skip to content

Fix hydrate-local-settings next-step hint to match README workflow#12

Closed
paulyuk wants to merge 1 commit into
mainfrom
paulyuk/fix-hydrate-next-step
Closed

Fix hydrate-local-settings next-step hint to match README workflow#12
paulyuk wants to merge 1 commit into
mainfrom
paulyuk/fix-hydrate-next-step

Conversation

@paulyuk

@paulyuk paulyuk commented Jun 16, 2026

Copy link
Copy Markdown
Member

What

hydrate-local-settings.{sh,ps1} prints Run 'func start' to start the host. as its final line. That's incomplete — the README's actual local-run workflow needs three terminals:

azurite --silent --skipApiVersionCheck --location .azurite   # terminal A
uv run func start                                            # terminal B
uv run python chat.py                                        # terminal C

Users following the script's hint get stuck — Azurite isn't running (bindings fail with 'No connection could be made'), and bare func start skips the uv run prefix so Core Tools picks the wrong Python interpreter.

Why

Caught while running through a fresh ~/src/m365-test1 clone end-to-end post-azd provision. The script tells the user one thing; the README tells them another. The script wins because it's the last thing they saw.

Change

Update both the bash and PowerShell variants to echo the three-terminal block verbatim from the README, so users land on the working path without re-reading docs.

Test

  • bash -n infra/scripts/hydrate-local-settings.sh passes
  • Manually ran the patched bash script in a hydrated env; output now matches the README quickstart block.
  • No behavior change beyond the final echo block.

The script printed 'Run func start to start the host', which is
incomplete and skips the actual local-run workflow documented in the
README:

  azurite --silent --skipApiVersionCheck --location .azurite  # terminal A
  uv run func start                                           # terminal B
  uv run python chat.py                                       # terminal C

Updated both the bash and PowerShell variants to print the three-terminal
block so users land on the working path without re-reading the README.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@paulyuk

paulyuk commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

Consolidated into a single PR per author preference: https://github.com/Azure-Samples/m365-inbox-serverless-agent-python/pull/(see latest)

@paulyuk paulyuk closed this Jun 16, 2026
@paulyuk paulyuk deleted the paulyuk/fix-hydrate-next-step branch June 16, 2026 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant