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 <noreply@continue.dev>
@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
Generated with [Continue](https://continue.dev)

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

Co-Authored-By: Continue <noreply@continue.dev>
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 <noreply@continue.dev>
Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <noreply@continue.dev>
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 <bdougie@users.noreply.github.com>
- 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 <noreply@continue.dev>
Co-authored-by: Username <bekah@continue.dev>
uinstinct and others added 8 commits October 10, 2025 05:33
- 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.

@rossigee rossigee requested a review from RomneyDa October 18, 2025 07:18
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.

@rossigee went over this with @TyDunn again today, sorry for the back and forth but we're thinking the slash command doesn't quite feel right. We could build a TUI menu where users could easily toggle but it doesn't solve CICD scenarios and the global context is a bit saturated (I know I requested putting it there).

We're thinking let's rip out the persisted setting altogether and just use an environment variable like CONTINUE_CLI_DISABLE_COMMIT_SIGNATURE. Thoughts on this?

@rossigee
Copy link
Contributor Author

That's a much better, simpler approach.

#8492

@rossigee rossigee closed this Oct 28, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Oct 28, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 28, 2025
@rossigee rossigee deleted the feature/disable-commit-signature-config branch October 30, 2025 00:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants