Skip to content

Add daily GitHub repo digest Functions sample#2

Open
paulyuk wants to merge 13 commits into
mainfrom
paulyuk-daily-github-repo-digest
Open

Add daily GitHub repo digest Functions sample#2
paulyuk wants to merge 13 commits into
mainfrom
paulyuk-daily-github-repo-digest

Conversation

@paulyuk

@paulyuk paulyuk commented Jul 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Update the sample concept to a daily GitHub repo digest for Azure/azure-functions-host
  • Preserve Azure Functions hosting, function_app.py, host.json, requirements.txt, and Copilot SDK runtime conventions
  • Add a timer-triggered daily digest path for 9 AM Pacific using public GitHub REST APIs
  • Keep the existing /api/ask chat/API behavior and update docs plus local config examples
  • Keep the main README focused on clone, restore, run, and azd up; move MCP extension notes into MCP-extension-notes.md

Validation

  • uv sync
  • uv run python -m py_compile function_app.py chat.py
  • Public GitHub REST API smoke check for Azure/azure-functions-host
  • Direct smoke test of _repo_digest_context('Azure/azure-functions-host') / MCP tool payload path with local imports

Paul Yuknewicz and others added 12 commits July 9, 2026 01:48
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

@lilyjma lilyjma left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

A couple of small fixes for the _run_digest function.

Comment thread function_app.py Outdated
Comment thread function_app.py Outdated
@lilyjma

lilyjma commented Jul 9, 2026

Copy link
Copy Markdown

Suggest removing local.settings.json from .gitignore. Including a default version in the repo makes local development smoother — users can clone and run immediately without being prompted about the stack they need. More importantly, the file created from the prompt is missing AzureWebJobsStorage, so running the app would error.

{
  "IsEncrypted": false,
  "Values": {
    "AzureWebJobsStorage": "UseDevelopmentStorage=true",
    "FUNCTIONS_WORKER_RUNTIME": "python"
  }
}

@lilyjma lilyjma left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggest adding a step here to start Azurite before running the function, since it's required for local storage emulation (AzureWebJobsStorage):

3. Start Azurite (in a separate terminal):

azurite --skipApiVersionCheck --silent --location ./.azurite

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

paulyuk commented Jul 9, 2026

Copy link
Copy Markdown
Member Author

Addressed Lily's feedback in e4e0f0d: fixed the Copilot SDK call shape, changed session cleanup to disconnect(), committed a default local.settings.json with AzureWebJobsStorage, and added the Azurite local-dev step.

@paulyuk paulyuk requested a review from lilyjma July 9, 2026 23:27
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.

3 participants