Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

Removes the discoverFromMemory function and its fallback invocation from campaign discovery. This function read worker cache files directly from /tmp/gh-aw/cache-memory/ as a last-resort discovery mechanism.

Changes

  • Removed discoverFromMemory function (117 lines)
  • Removed fallback block that invoked it when label-based discovery returned no items
  • Updated comment referencing cache discovery

Discovery flow after this change

Campaign discovery now follows this sequence:

  1. Campaign-specific label search (z_campaign_{id})
  2. Generic agentic-campaign label search
  3. GitHub API search by tracker-id
  4. Custom tracker label (if configured)

The function was already marked DEPRECATED and was never exported from the module.

Original prompt

Remove campaigns DEPRECATED cache-memory discovery.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove campaigns deprecated cache-memory discovery Remove deprecated cache-memory discovery from campaigns Jan 22, 2026
Copilot AI requested a review from mnkiefer January 22, 2026 16:15
@mnkiefer
Copy link
Contributor

@copilot Recompile workflows

Copy link
Contributor Author

Copilot AI commented Jan 22, 2026

@copilot Recompile workflows

Recompiled workflows successfully. All 134 workflow lock files are up to date - no changes were needed.

Copilot AI requested a review from mnkiefer January 22, 2026 16:32
@mnkiefer mnkiefer marked this pull request as ready for review January 22, 2026 16:35
@mnkiefer mnkiefer merged commit adb594b into main Jan 22, 2026
49 checks passed
@mnkiefer mnkiefer deleted the copilot/remove-deprecated-cache-memory branch January 22, 2026 16:53
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.

2 participants