Skip to content

Comments

refactor: extract listener-core as standalone crate#4172

Merged
yujonglee merged 5 commits intomainfrom
refactor/extract-listener-core
Feb 23, 2026
Merged

refactor: extract listener-core as standalone crate#4172
yujonglee merged 5 commits intomainfrom
refactor/extract-listener-core

Conversation

@yujonglee
Copy link
Contributor

Summary

  • Moves all actor logic (listener, recorder, session, source) out of the plugins/listener Tauri plugin into a new platform-agnostic crates/listener-core crate
  • The Tauri plugin now depends on listener-core rather than owning the implementation
  • Adds a cli example in listener-core for testing the core logic without Tauri

Motivation

Decoupling the core listener logic from the Tauri plugin layer makes it easier to test, reuse, and develop independently of the desktop app runtime.

Made with Cursor

yujonglee and others added 2 commits February 23, 2026 10:48
Move actor logic out of the Tauri plugin into a platform-agnostic
`listener-core` crate, enabling reuse without Tauri dependencies.

Co-authored-by: Cursor <cursoragent@cursor.com>
@netlify
Copy link

netlify bot commented Feb 23, 2026

Deploy Preview for hyprnote-storybook canceled.

Name Link
🔨 Latest commit 6d1b953
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/699bc906c6db0d000895a53c

@netlify
Copy link

netlify bot commented Feb 23, 2026

Deploy Preview for hyprnote canceled.

Name Link
🔨 Latest commit 6d1b953
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/699bc906ac9b2c00086ef918

…ener-core

Co-authored-by: Cursor <cursoragent@cursor.com>

# Conflicts:
#	packages/transcript/src/pass-build-segments.ts
@yujonglee yujonglee merged commit cefe001 into main Feb 23, 2026
15 checks passed
@yujonglee yujonglee deleted the refactor/extract-listener-core branch February 23, 2026 03:27
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

continue;
}

let name = path.file_name()?.to_str()?;
Copy link

Choose a reason for hiding this comment

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

Early return from ? aborts directory search prematurely

Low Severity

In find_session_dir_recursive, the ? operator on path.file_name()?.to_str()? inside the for loop will short-circuit and return None from the entire function if any directory entry has a non-UTF-8 name, rather than just skipping that entry. This causes the search to abort prematurely, and the caller falls back to the default path, potentially missing the real session directory. Using a let-else with continue would correctly skip the problematic entry.

Fix in Cursor Fix in Web

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.

1 participant