Skip to content

fix(deep-report): switch to claude engine and fix memory path#23807

Merged
pelikhan merged 3 commits intomainfrom
copilot/fix-deep-report-switch-to-claude-engine
Apr 1, 2026
Merged

fix(deep-report): switch to claude engine and fix memory path#23807
pelikhan merged 3 commits intomainfrom
copilot/fix-deep-report-switch-to-claude-engine

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 1, 2026

The DeepReport workflow was failing due to OpenAI's safety classifier blocking cybersecurity-adjacent content, and agent memory writes were silently discarded due to a wrong path in the prompt instructions.

Changes

  • Engine: codexclaude — eliminates recurring OpenAI blocks triggered by phrases like "Detect suspicious patterns" / "Suspicious activity"
  • Memory path (Step 0 + Step 4): /tmp/gh-aw/repo-memory-default/memory/default//tmp/gh-aw/repo-memory/default/memory/deep-report/ — aligns with the compiled lock file's GH_AW_MEMORY_DIR and FILE_GLOB_FILTER, so insights are actually persisted and reused across runs

Copilot AI and others added 2 commits April 1, 2026 02:32
Agent-Logs-Url: https://github.com/github/gh-aw/sessions/25965d10-64b5-4784-b7b3-c0ab26920b6d

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix deep report to switch to Claude engine and resolve memory path fix(deep-report): switch to claude engine and fix memory path Apr 1, 2026
Copilot AI requested a review from pelikhan April 1, 2026 02:37
@pelikhan pelikhan marked this pull request as ready for review April 1, 2026 02:38
Copilot AI review requested due to automatic review settings April 1, 2026 02:38
@pelikhan pelikhan merged commit 8d635a7 into main Apr 1, 2026
57 checks passed
@pelikhan pelikhan deleted the copilot/fix-deep-report-switch-to-claude-engine branch April 1, 2026 02:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Switches the DeepReport agent workflow from OpenAI Codex to Anthropic Claude Code to avoid recurring safety-classifier blocks, and corrects the repo-memory path so persisted insights are actually reused across runs.

Changes:

  • Update DeepReport workflow engine from codex to claude (including secret validation, CLI install/exec, log parsing, and concurrency naming).
  • Fix DeepReport repo-memory instructions to use /tmp/gh-aw/repo-memory/default/memory/deep-report/ (matching the workflow’s repo-memory configuration).
  • Bump the pinned github/gh-aw-actions/setup action version and extend actions lock metadata for artifact actions.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
pkg/workflow/data/action_pins.json Bumps github/gh-aw-actions/setup pin to v0.65.2 (new SHA).
.github/workflows/deep-report.md Switches engine to Claude and corrects repo-memory read/write path in the prompt instructions.
.github/workflows/deep-report.lock.yml Updates the compiled workflow to run Claude Code CLI (secrets, parsing, domains, concurrency, MCP config usage).
.github/aw/actions-lock.json Adds lock entries for actions/download-artifact@v4 and actions/upload-artifact@v4.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

[q] fix(deep-report): switch to claude engine and fix memory path (#23140)

3 participants