Skip to content

Conversation

etherandrius
Copy link

@etherandrius etherandrius commented Aug 22, 2025

Description

#7337

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

Summary by cubic

Removed completionOptions from PromptLog and dropped appliedRules tracking to simplify logs and shrink stored session data. Updated core LLM logging, Redux state/actions, and tests to match the new shape.

  • Refactors
    • core: Removed PromptLog.completionOptions and its use in BaseLLM; updated DUD_PROMPT_LOG.
    • gui: Removed setAppliedRulesAtIndex and appliedRules persistence; stopped attaching tools/rules metadata in streamNormalInput.
    • tests: Updated to reflect removed fields and actions.

@etherandrius etherandrius requested a review from a team as a code owner August 22, 2025 17:35
@etherandrius etherandrius requested review from RomneyDa and removed request for a team August 22, 2025 17:35
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 22, 2025
Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


agrabauskas seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@@ -456,7 +456,6 @@ export type FileSymbolMap = Record<string, SymbolWithRange[]>;
export interface PromptLog {
modelTitle: string;
modelProvider: string;
completionOptions: CompletionOptions;
Copy link
Author

Choose a reason for hiding this comment

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

Main change everything else is cleanup.

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Aug 22, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Aug 27, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 27, 2025
@RomneyDa RomneyDa closed this Aug 27, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Aug 27, 2025
@RomneyDa RomneyDa reopened this Aug 27, 2025
@github-project-automation github-project-automation bot moved this from Done to In Progress in Issues and PRs Aug 27, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Aug 27, 2025
@RomneyDa
Copy link
Collaborator

@etherandrius this looks good to me! Looks like waiting on CLA signing

@RomneyDa
Copy link
Collaborator

or maybe CLA bot glitched, rerunning

@RomneyDa RomneyDa merged commit e58b250 into continuedev:main Aug 27, 2025
88 of 91 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Aug 27, 2025
@sestinj
Copy link
Contributor

sestinj commented Sep 1, 2025

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Sep 3, 2025

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm This PR has been approved by a maintainer released size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants