Skip to content
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

[bounty] switch pipes engine from deno to bun #582

Closed
louis030195 opened this issue Oct 25, 2024 · 7 comments
Closed

[bounty] switch pipes engine from deno to bun #582

louis030195 opened this issue Oct 25, 2024 · 7 comments
Labels
💎 Bounty enhancement New feature or request

Comments

@louis030195
Copy link
Collaborator

definition of done:

  • all pipes works with new engine (updated if necessary), including the screenpipe-js sdk
  • pipes works on all OS - (you tested it, i can test on macos, i cannot test for windows/linux)
  • meaning you need to update how we package deno to bun on different OS

why? nextjs does not work well in deno, too many issues, not many upsides (for example https://github.com/vercel/ai-chatbot was crashing in deno when asking AI questions, e.g. SSE not supported or something)

/bounty 100

@louis030195 louis030195 added the enhancement New feature or request label Oct 25, 2024
Copy link

linear bot commented Oct 25, 2024

Copy link

algora-pbc bot commented Oct 25, 2024

💎 $100 bounty • Screenpi.pe

Steps to solve:

  1. Start working: Comment /attempt #582 with your implementation plan
  2. Submit work: Create a pull request including /claim #582 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to mediar-ai/screenpipe!

Add a bountyShare on socials

Attempt Started (GMT+0) Solution
🟢 @kumarvivek1752 Oct 26, 2024, 10:11:47 AM WIP
🔴 @Abiji-2020 Oct 26, 2024, 2:25:06 PM WIP

@kumarvivek1752
Copy link

kumarvivek1752 commented Oct 26, 2024

/attempt #582

@Abiji-2020
Copy link

https://github.com/mediar-ai/screenpipe/blob/main/screenpipe-core/src/pipes.rs

In this we have to implement the bun right from deno

@Abiji-2020
Copy link

Abiji-2020 commented Oct 26, 2024

/attempt #582

Algora profile Completed bounties Tech Active attempts Options
@Abiji-2020 2 bounties from 2 projects
C++, Java,
TypeScript & more
Cancel attempt

@kumarvivek1752
Copy link

@louis030195 I added configuration for Bun, and I'm testing the plugin with Bun.

demo bunfig.toml:

[dependencies]
screenpipe = { url = "https://raw.githubusercontent.com/mediar-ai/screenpipe/main/screenpipe-js/main.ts" }

[imports]
fs = "node:fs"
screenpipe = "https://raw.githubusercontent.com/mediar-ai/screenpipe/main/screenpipe-js/main.ts"

pipe.ts:

import {
  queryScreenpipe,
  loadPipeConfig,
  ContentItem,
  extractJsonFromLlmResponse,
} from "screenpipe";

console.log("Loading pipe config:", loadPipeConfig());
console.log("Plugin is running");

when I try to use screenpipe package

➜ screenpipe git:(main) bun run --allow-read --allow-write --allow-env --allow-net --allow-run --allow-sys .\examples\typescript\pipe-meeting-summary-by-email\pipe.ts
error: Cannot find package "screenpipe" from "C:\Users\kumar\devlopment\screenpipe\examples\typescript\pipe-meeting-summary-by-email\pipe.ts"

Bun v1.1.29 (Windows x64)
➜ screenpipe git:(main) ✗

am I missing something?

@louis030195
Copy link
Collaborator Author

i solved this issue yesterday, sorry was too slow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💎 Bounty enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants