Skip to content

DeepReport: align repo-memory paths with branch root and allow JSON state files - #51931

Merged
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-fix-repo-memory-path
Aug 11, 2026
Merged

DeepReport: align repo-memory paths with branch root and allow JSON state files#51931
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-fix-repo-memory-path

Conversation

Copilot AI commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

DeepReport was writing/reading repo-memory under a non-matching two-level path (.../default/memory/deep-report/...) while the configured glob only matched files at the repo-memory root scope used by the workflow. It also excluded required JSON state artifacts (processed-discussions.json, extracted-tasks.json) from repo-memory writes.

  • Frontmatter changes

    • Updated tools.repo-memory.file-glob from ["*.md"] to ["*.md", "*.json"].
  • Prompt path corrections

    • Repointed DeepReport repo-memory read/write paths to:
      • /tmp/gh-aw/repo-memory/default/deep-report/
    • Updated mining step references to explicit JSON paths under that directory.
  • Instruction consistency

    • Adjusted prompt text to reflect that both Markdown and JSON files are part of the repo-memory contract for this workflow.
  • Compiled artifact refresh

    • Regenerated .github/workflows/deep-report.lock.yml to reflect updated glob constraints and memory filter values.
tools:
  repo-memory:
    branch-name: memory/deep-report
    file-glob: ["*.md", "*.json"]

# Prompt paths now use:
# /tmp/gh-aw/repo-memory/default/deep-report/
# including processed-discussions.json and extracted-tasks.json

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix DeepReport repo-memory path issue from #51172 DeepReport: align repo-memory paths with branch root and allow JSON state files Aug 11, 2026
Copilot AI requested a review from pelikhan August 11, 2026 04:51
@pelikhan
pelikhan marked this pull request as ready for review August 11, 2026 04:59
Copilot AI balanced review requested due to automatic review settings August 11, 2026 04:59
@pelikhan
pelikhan merged commit 3afb27f into main Aug 11, 2026
@pelikhan
pelikhan deleted the copilot/deep-report-fix-repo-memory-path branch August 11, 2026 04:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes #51809 by aligning DeepReport’s repo-memory paths and permitted file types with runtime filtering.

Changes:

  • Moves memory access to the correct default/deep-report/ directory.
  • Allows Markdown and JSON state files.
  • Regenerates the compiled workflow.
Show a summary per file
File Description
.github/workflows/deep-report.md Corrects paths and permits JSON memory files.
.github/workflows/deep-report.lock.yml Refreshes generated metadata and memory filters.

Review details

Tip

Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.86.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants