Skip to content

Add configurable path for job result messages separate from workers directory#51

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/separate-job-result-directory
Draft

Add configurable path for job result messages separate from workers directory#51
Copilot wants to merge 4 commits intomainfrom
copilot/separate-job-result-directory

Conversation

Copy link
Contributor

Copilot AI commented Nov 22, 2025

Job result messages were hardcoded to {WorkerPath}/__messages__/, coupling message storage to the workers directory.

Changes

  • Configuration: Added JobResultMessagesPath setting to decouple message storage location
  • API: Updated IJobManager methods to accept jobResultMessagesPath instead of deriving from workerPath
  • Migration: Enhanced MigrateDetailedMessages with fallback to {WorkerPath}/__messages__ for backward compatibility

Usage

{
  "WorkerPath": "C:/temp/workers",
  "JobResultMessagesPath": "C:/temp/job-messages"
}

Messages now write directly to JobResultMessagesPath instead of a subdirectory under workers.

Original prompt

This section details on the original issue you should resolve

<issue_title>Separate configured location for job result messages instead of putting it within the workers directory</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Nov 22, 2025
Copilot AI and others added 3 commits November 22, 2025 17:57
…from workers directory

Co-authored-by: jtenos <607782+jtenos@users.noreply.github.com>
Co-authored-by: jtenos <607782+jtenos@users.noreply.github.com>
Co-authored-by: jtenos <607782+jtenos@users.noreply.github.com>
Copilot AI changed the title [WIP] Separate configured location for job result messages Add configurable path for job result messages separate from workers directory Nov 22, 2025
Copilot AI requested a review from jtenos November 22, 2025 18:03
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.

Separate configured location for job result messages instead of putting it within the workers directory

2 participants