Skip to content

Improve error for nonexistent history source manifest paths#49

Merged
luca-ctx merged 2 commits into
ctxrs:mainfrom
atharva-patill:fix/history-source-manifest-path
Jul 4, 2026
Merged

Improve error for nonexistent history source manifest paths#49
luca-ctx merged 2 commits into
ctxrs:mainfrom
atharva-patill:fix/history-source-manifest-path

Conversation

@atharva-patill

Copy link
Copy Markdown
Contributor

Summary

The --history-source-manifest import path currently reports that the supplied path "did not contain" ctx-history-plugin.json even when the path does not exist.

This adds the same early path validation already used by the other explicit import paths, so nonexistent paths report the existing import path does not exist error instead.

Reproduction

./target/debug/ctx import \
  --history-source-manifest /tmp/no-such-manifest.json

Validation

  • Added a regression test for nonexistent manifest paths
  • Verified the new behavior manually

Before

beforectx

After

afterctx

@luca-ctx luca-ctx merged commit 98f722f into ctxrs:main Jul 4, 2026
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