Skip to content

Conversation

@oswaldo
Copy link
Owner

@oswaldo oswaldo commented Dec 9, 2025

Implements the update command and adds includes support to save. See walkthrough for details.

@estelamaris estelamaris self-requested a review December 10, 2025 10:17
Copilot AI review requested due to automatic review settings December 10, 2025 22:59
Copy link

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

estelamaris
estelamaris previously approved these changes Dec 10, 2025
estelamaris
estelamaris previously approved these changes Dec 10, 2025
Copilot AI review requested due to automatic review settings December 10, 2025 23:38
Copy link

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@oswaldo oswaldo disabled auto-merge December 10, 2025 23:48
@oswaldo oswaldo force-pushed the 008-update-fctx branch 2 times, most recently from 8d0d41b to 5fe3a30 Compare December 11, 2025 00:26
Copilot AI review requested due to automatic review settings December 11, 2025 00:26
Copy link

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

Copilot reviewed 30 out of 31 changed files in this pull request and generated 2 comments.


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

- Implements `UpdateCommand` with options for adding/removing artifacts and includes.
- Enhances `SaveCommand` to support specifying `--includes`.
- Refactors artifact processing into reusable `ArtifactProcessor`.
- Implements `FctxWriter` for robust HOCON generation preserving includes.
- Adds comprehensive tests in `UpdateTests`, `SaveTests`, `ArtifactProcessorTests`, and `FctxWriterTests`.
- Updates documentation and verifies via manual walkthrough.
@oswaldo oswaldo merged commit 7a22a1d into main Dec 11, 2025
1 check passed
@oswaldo oswaldo deleted the 008-update-fctx branch December 11, 2025 01:01
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