Skip to content

Conversation

@benfaerber
Copy link
Contributor

What does this PR do?

I was confused how to set the API key I tried a bunch of different key names like: APIKey, grokApiKey, etc.

Fixes #
This just tells the user the key name to set to make it a bit easier.

Checklist

  • I tested my changes
  • I reviewed my own code

@homanp
Copy link
Contributor

homanp commented Nov 27, 2025

@benfaerber Thanks for this, merging!

@homanp homanp merged commit debe6f9 into superagent-ai:main Nov 27, 2025
2 checks passed
homanp added a commit that referenced this pull request Nov 27, 2025
* feat: add support for global custom instructions

* Remove comments

* Fix import syntax for fs-extra module (#112)

* Fix import syntax for fs-extra module

* Fix import syntax for fs-extra module

* Fix import syntax for fs-extra module

* Fix missing newline at end of morph-editor.ts

* Add where to set the API key in the json file (#107)

* Fix diff generation showing entire file as changed (fixes #83) (#98)

The diff preview was showing the entire file as changed when only a few
lines were actually modified. This was caused by a broken change detection
algorithm that couldn't properly handle insertions and deletions.

Replaced the naive line-matching algorithm with a proper LCS (Longest
Common Subsequence) based diff algorithm using dynamic programming. This
produces minimal, accurate diffs that match what git diff shows.

Changes:
- Added computeLCS(): Computes longest common subsequence between old/new lines
- Added extractChanges(): Extracts actual change regions from LCS table
- Updated generateDiff(): Uses LCS-based change detection instead of broken algorithm

Impact:
- Diff previews now show only actual changes (e.g., 4 lines instead of 260)
- Drastically reduces context pollution in conversation history
- Matches git diff output accuracy

🤖 Contributed by ZDS-AI (https://zds.group)

* add model migration manager and new grok models (#124)

* minor tweaks

---------

Co-authored-by: jszczypk <[email protected]>
Co-authored-by: Ben Faerber <[email protected]>
Co-authored-by: Joseph Cheek <[email protected]>
Co-authored-by: Ismail Pelaseyed <[email protected]>
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.

2 participants