Skip to content

Conversation

rossigee
Copy link
Contributor

@rossigee rossigee commented Oct 8, 2025

Description

Added a new configuration option disableCommitSignature to allow users to disable the automatic inclusion of Continue attribution in commit messages generated by the CLI. This makes the signature configurable for users who prefer not to include it.

Key changes:

  • Updated config types and YAML loading to support the new disableCommitSignature boolean option
  • Modified the CLI system message to conditionally include the signature based on the config
  • Added comprehensive tests for the new behavior
  • Updated documentation in the config reference

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

[Not applicable for this configuration change.]

Tests

Added tests in extensions/cli/src/systemMessage.test.ts to verify the signature is included or excluded based on the disableCommitSignature config option.


Summary by cubic

Made the Continue commit attribution optional for CLI-generated commits via a new disableCommitSignature config. By default the signature is included; set disableCommitSignature: true to remove it.

  • New Features

    • Added disableCommitSignature to config types and YAML loading.
    • System message now includes the commit signature only when not disabled.
    • Updated docs with a usage example.
  • Migration

    • To disable, add disableCommitSignature: true to config.yaml.

Add disableCommitSignature config option to allow users to disable
the automatic inclusion of Continue attribution in commit messages.

- Add disableCommitSignature to config interfaces
- Update system message to conditionally include signature
- Add comprehensive tests for the new behavior
- Update documentation in config.yaml reference

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
@rossigee rossigee requested a review from a team as a code owner October 8, 2025 22:25
@rossigee rossigee requested review from tingwai and removed request for a team October 8, 2025 22:25
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 8, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

rossigee and others added 4 commits October 9, 2025 05:57
The disableCommitSignature property was added to the AssistantUnrolled schema but the package build was failing due to missing @continuedev/config-types dependency. Installed the dependency and rebuilt the package to update TypeScript types.

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
Copy link
Collaborator

@RomneyDa RomneyDa left a comment

Choose a reason for hiding this comment

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

Requesting changes to delay merging as we don't currently top level settings like this to config yaml

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Oct 9, 2025
@RomneyDa
Copy link
Collaborator

RomneyDa commented Oct 9, 2025

@rossigee currently settings like this live in GlobalContext

tingwai and others added 15 commits October 10, 2025 05:31
* Add notion cookbook

* Add notion draft

* Add draft notion cookbook

* Update docs/guides/notion-continue-cookbook.mdx

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

* Change cookbook to guide using API since MCP doesn't work

* Move out of MCP cookbooks in overview

* Apply suggestion from @cubic-dev-ai[bot]

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>

---------

Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Co-authored-by: Brian Douglas <[email protected]>
- Added dismissable banner to docs.json
- Banner promotes CLI v1.4.49 release and links to MCP Cookbooks
- Banner is dismissable for better UX

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
Co-authored-by: Username <[email protected]>
tomasz-stefaniak and others added 24 commits October 10, 2025 05:31
- Move disableCommitSignature from config.yaml to GlobalContext for user preferences
- Add /set slash command for configuring CLI preferences
- Update documentation to reflect new user-configurable setting
- Add comprehensive tests for the new functionality
- Maintain backward compatibility with default attribution enabled

The setting is now stored in GlobalContext and can be configured using:
- /set disableCommitSignature true (disable attribution)
- /set disableCommitSignature false (enable attribution, default)
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Oct 9, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Oct 9, 2025
@RomneyDa
Copy link
Collaborator

RomneyDa commented Oct 13, 2025

@rossigee appreciate the rework, implementation lgtm, I'll run this past @TyDunn, we might want to tweak the slash command naming from /set but otherwise looks good.

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

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

7 participants