fix: warn on malformed ~/.klaatai/config.json - #11
Closed
Devil1716 wants to merge 1 commit into
Closed
Conversation
Malformed config previously fell back to defaults with no signal. Print a one-line stderr warning naming the file and parse error, then continue. Co-authored-by: Cursor <cursoragent@cursor.com>
Member
|
@Devil1716 can you please try to attach a screenshot with your comment. |
Contributor
Author
|
Hi Aakash - thanks for taking a look.
I saw #11 was closed before I could attach the screenshot. Happy to open a
follow-up PR (or reopen) with a clear screenshot of the malformed-config
warning if that would still help.
Thanks again!
…On Sat, 18 Jul 2026 at 13:04, Aakash Singh ***@***.***> wrote:
*aakashrajput* left a comment (KlaatAI/klaatcode#11)
<#11 (comment)>
@Devil1716 <https://github.com/Devil1716> can you please try to attach a
screenshot with your comment.
—
Reply to this email directly, view it on GitHub
<#11?email_source=notifications&email_token=BDWREBQL5JDIGJNVROH5S735FMSBFA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMBRGA2DGNBSGY42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5010434269>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BDWREBR56N6EKJ4WAQV3T5D5FMSBFAVCNFSNUABGKJSXA33TNF2G64TZHMYTGMBTHEZDSMJVGI5US43TOVSTWNBZGE3TEOBWGE3TTILWAI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Member
|
@Devil1716 you are always welcome to do improvement in the project. If you have idea or a better improvement open a issue and work on it. also you can join the discord (link can be found on readme) and use Contribution channel to ask to create issue and assign to you. Happy Coding. |
Contributor
Author
|
Thanks Floze - that matrix breakdown was really helpful so the feather/ipc
change wasn't blocked by opaque CI failures. Appreciate you digging into
the logs!
…On Mon, 20 Jul 2026 at 19:25, Aakash Singh ***@***.***> wrote:
*aakashrajput* left a comment (KlaatAI/klaatcode#11)
<#11 (comment)>
@Devil1716 <https://github.com/Devil1716> you are always welcome to do
improvement in the project. If you have idea or a better improvement open a
issue and work on it. also you can join the discord (link can be found on
readme) and use Contribution channel to ask to create issue and assign to
you.
Happy Coding.
—
Reply to this email directly, view it on GitHub
<#11?email_source=notifications&email_token=BDWREBSBP3W6IGJN47PZYWL5FYQGPA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMBSGMYDINRUGI42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5023046429>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BDWREBTWRKYJZ6TYUQJPGRT5FYQGPAVCNFSNUABGKJSXA33TNF2G64TZHMYTGMBTHEZDSMJVGI5US43TOVSTWNBZGE3TEOBWGE3TTILWAI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Contributor
Author
|
Thanks Mathusan - really appreciate the merge on #6! Glad the TfL hub
expansion into station children made sense.
…On Mon, 20 Jul 2026 at 19:25, Aakash Singh ***@***.***> wrote:
*aakashrajput* left a comment (KlaatAI/klaatcode#11)
<#11 (comment)>
@Devil1716 <https://github.com/Devil1716> you are always welcome to do
improvement in the project. If you have idea or a better improvement open a
issue and work on it. also you can join the discord (link can be found on
readme) and use Contribution channel to ask to create issue and assign to
you.
Happy Coding.
—
Reply to this email directly, view it on GitHub
<#11?email_source=notifications&email_token=BDWREBSBP3W6IGJN47PZYWL5FYQGPA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMBSGMYDINRUGI42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5023046429>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BDWREBTWRKYJZ6TYUQJPGRT5FYQGPAVCNFSNUABGKJSXA33TNF2G64TZHMYTGMBTHEZDSMJVGI5US43TOVSTWNBZGE3TEOBWGE3TTILWAI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Contributor
Author
|
Sorry about the two off-topic comments just now - those were meant for
other project threads and got sent here by mistake. Please ignore them. The
earlier note about the screenshot still stands. Thanks!
…On Mon, 20 Jul 2026 at 19:26, Sharan ***@***.***> wrote:
Thanks Mathusan - really appreciate the merge on #6! Glad the TfL hub
expansion into station children made sense.
On Mon, 20 Jul 2026 at 19:25, Aakash Singh ***@***.***>
wrote:
> *aakashrajput* left a comment (KlaatAI/klaatcode#11)
> <#11 (comment)>
>
> @Devil1716 <https://github.com/Devil1716> you are always welcome to do
> improvement in the project. If you have idea or a better improvement open a
> issue and work on it. also you can join the discord (link can be found on
> readme) and use Contribution channel to ask to create issue and assign to
> you.
>
> Happy Coding.
>
> —
> Reply to this email directly, view it on GitHub
> <#11?email_source=notifications&email_token=BDWREBSBP3W6IGJN47PZYWL5FYQGPA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMBSGMYDINRUGI42M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5023046429>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/BDWREBTWRKYJZ6TYUQJPGRT5FYQGPAVCNFSNUABGKJSXA33TNF2G64TZHMYTGMBTHEZDSMJVGI5US43TOVSTWNBZGE3TEOBWGE3TTILWAI>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
loadConfig()still falls back to defaults on parse failure, but now writes a one-line stderr warning with the file path and error.Test plan
~/.klaatai/config.json(trailing comma) and runklaatcode— see warningFixes #8
Note: issue is currently assigned to @itxhadi27-cmd — happy to close/defer if they already have a PR in progress.