Skip to content

Conversation

shaik-zeeshan
Copy link
Contributor

@shaik-zeeshan shaik-zeeshan commented May 23, 2025

desktop app of rewind pipe

openrewind.1.mp4

Copy link
Contributor

🧪 testing bounty created!

a testing bounty has been created for this PR: view testing issue

testers will be awarded $20 each for providing quality test reports. please check the issue for testing requirements.

@louis030195
Copy link
Collaborator

how can i test it?

@shaik-zeeshan
Copy link
Contributor Author

shaik-zeeshan commented May 28, 2025

issue was happening randomly. the settings were not loaded from file. so it was showing default settings. to test try reloading page or relaunching app again again. if you see the settings in app are reset(don't change anything in settings). check the settings file. settings will still be there

@shaik-zeeshan
Copy link
Contributor Author

@louis030195 Will the app be completely standalone, or will it share data with screenpipe-app? For example, if I have both screenpipe-app and rewind-app running on my PC, will the recording, resource and data be shared between them, or will they operate independently?

@kush34
Copy link

kush34 commented May 29, 2025

Hey, is testing still required? i can help

@louis030195
Copy link
Collaborator

@louis030195 Will the app be completely standalone, or will it share data with screenpipe-app? For example, if I have both screenpipe-app and rewind-app running on my PC, will the recording, resource and data be shared between them, or will they operate independently?

hmm

i think it would make sense to be shared

tricky part is to check to avoid conflict, having two times the recorder in parallel

@louis030195
Copy link
Collaborator

Hey, is testing still required? i can help

later probably yes

@louis030195
Copy link
Collaborator

also wdyt of making this in the same folder than the existing tauri app, but during build time it would enable / disable specific feature in rust side / js side / show different UI?

been considering this path too

and in CI/cd it would build 2 apps

one advantage is to keep the same code for rewind and stuff, not sure easier to maintain?

@shaik-zeeshan
Copy link
Contributor Author

it could get more complex and code will be messy since we mixing two apps, also there is risk of breaking one app while working on another.

@shaik-zeeshan shaik-zeeshan marked this pull request as ready for review June 16, 2025 17:05
@louis030195
Copy link
Collaborator

Screenshot 2025-06-18 at 6 10 24 PM Screenshot 2025-06-18 at 6 10 20 PM

somehow super zoomed in

@louis030195
Copy link
Collaborator

there are a few issues in the onboarding but i think its same one known in screenpipe such as:

  • most permissions randomly pop up at wrong timing (macos)
  • screen stream does not show up
  • still facing issues with AI presets but idk
  • shortcuts show "Super" which is weird on Mac (should be cmd)

overall its really great job, would love to ship this to all - also it's rly cool it can use the existing screenpipe data

@louis030195
Copy link
Collaborator

Screenshot 2025-06-19 at 4 48 44 PM

not sure what changed, build crash for me now (when i open the app built with bun tauri build), maybe the window change

@shaik-zeeshan
Copy link
Contributor Author

shaik-zeeshan commented Jul 9, 2025

@louis030195 anything more to add to this pr? also could check if prev issues are fixed?

@louis030195
Copy link
Collaborator

louis030195 commented Jul 9, 2025

testing

for anyone wanting to install this app right now:

do this or paste in cursor chat to do it for you:

gh pr checkout 1805
cargo build --release
cd openrewind
bun i
bun tauri build
# open the end build .dmg path

@louis030195
Copy link
Collaborator

louis030195 commented Jul 9, 2025

still crashing for me at start

since around 196d53e

@shaik-zeeshan
Copy link
Contributor Author

shaik-zeeshan commented Jul 10, 2025

could try deleting existing app (settings,etc) and install the new one?

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.

3 participants