-
Notifications
You must be signed in to change notification settings - Fork 1
feat(cli): replace Docker-based get-sync-token with native Obsidian API call #492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
2a661e3
feat(cli): replace Docker-based get-sync-token with native Obsidian A…
aliasunder d939c87
fix(review): update stale Docker references in comments
aliasunder d920916
test: add MFA retry failure coverage and tighten assertion
aliasunder 7ce7b17
fix(cli): classify retry errors in MFA flow instead of blaming the 2F…
aliasunder d9b07b6
fix(cli): tighten retry error hint + non-object JSON guard; lead guid…
aliasunder 33b0ee5
docs: add Node.js version check step before npx in one-click guides
aliasunder 23189b3
docs: remove confusing Docker fallback reference from Node.js check step
aliasunder 36ee413
Merge branch 'main' into worktree-get-sync-token-native
aliasunder 99803e0
feat(cli): remove paste prompt, add orientation text for token capture
aliasunder 1ae3ec6
docs: remove "No Docker needed" from token capture steps
aliasunder 343b05c
docs(cli): simplify init's get-sync-token reference
aliasunder 2b7ef96
Merge branch 'main' into worktree-get-sync-token-native
aliasunder bbdbbc6
refactor(cli): rename isRecord to isJsonObject for clarity
aliasunder a15e4da
refactor(cli): improve get-sync-token readability
aliasunder d7f499b
refactor(cli): flatten callSigninApi response parsing
aliasunder 4c94e68
refactor(cli): wrap response parsing in try/catch
aliasunder c81c0a0
refactor(cli): simplify callSigninApi to return token string
aliasunder 2c4ba10
fix(cli): preserve existing token on re-init; simplify API return type
aliasunder 3674881
refactor(cli): use undefined for missing token instead of empty string
aliasunder 4e7ba9e
fix(cli): update PTY test for removed paste prompt + type obsidianAut…
aliasunder 10282b7
fix(cli): add missing tests, fix comment, make signin URL configurable
aliasunder 071e2c1
test(cli): add PTY happy-path tests for get-sync-token
aliasunder f2c5bc5
fix(cli): correct stale paste-prompt reference in offerSyncTokenCaptu…
aliasunder acf2e46
feat(cli): show start command after get-sync-token --dir writes token
aliasunder c90e598
fix(cli): remove redundant orientation text and merge start hint into…
aliasunder File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.