https://discord.cloudflare.com logo
Join Discord
Powered by
# workers-discussions
  • j

    James

    04/27/2023, 3:56 PM
    Tried this in my test unfortunately, still exceeds startup limits
  • k

    kian

    04/27/2023, 3:56 PM
  • k

    kian

    04/27/2023, 3:56 PM
    I guess there isn't anything you can do about, uh, this
  • t

    Tom Sherman

    04/27/2023, 3:57 PM
    i can't find that code in the repo, most be a dependency?
  • k

    kian

    04/27/2023, 3:57 PM
    probably apart of their xrpc stuff
  • k

    kian

    04/27/2023, 3:57 PM
    BskyAgent
    extends
    BaseAgent
    which constructs
    XrpcClient
    from` @atproto/xrpc`
  • k

    kian

    04/27/2023, 3:58 PM
    names aren't exact since I don't have it open
  • t

    Tom Sherman

    04/27/2023, 3:58 PM
    the
    xrpc
    package is in the repo tho, that code doesn't live there
  • k

    kian

    04/27/2023, 3:58 PM
    no idea then - but it gets bundled in
  • j

    James

    04/27/2023, 3:59 PM
    that looks like the
    tlds
    package to me
  • j

    James

    04/27/2023, 3:59 PM
    which is probably bundled in somewhere for link checking
  • t

    Tom Sherman

    04/27/2023, 3:59 PM
    remind me why they didn't use activity pub again? 😂
  • o

    Ottomated

    04/27/2023, 4:00 PM
    Cool, so is this correct? https://discord.com/channels/595317990191398933/779390076219686943/1101149926756470794 Or does Quicksilver store the external modules separately?
  • t

    Tom Sherman

    04/27/2023, 4:01 PM
    this needs to be a web proposal IMO, should be accessible on
    URL.tlds
    or something. crazy that each browser has to ship this list but it's not accessible to JS 😅
  • k

    kian

    04/27/2023, 4:01 PM
    No idea, that'd be an implementation detail which only an employee would be able to answer
  • j

    James

    04/27/2023, 4:01 PM
    that's a really good idea tbh
  • w

    wesbos

    04/27/2023, 4:02 PM
    I tried to take the TLDs out and it still didnt run
  • j

    James

    04/27/2023, 4:02 PM
    Yeah the code here is dense: https://github.com/bluesky-social/atproto/tree/main/packages They seem to be doing a lot of work on instantiation
  • j

    James

    04/27/2023, 4:03 PM
    Which makes a lot of sense in an environment like Node.js with long-running apps to save parsing time later. But less so on Workers 😦
  • t

    Tom Sherman

    04/27/2023, 4:03 PM
    "the lambda already takes 5s to startup, what's 500ms more!"
  • w

    wesbos

    04/27/2023, 4:06 PM
    shoot yeah, tree shaking doesnt do anything either
  • w

    wesbos

    04/27/2023, 4:07 PM
    maybe ill open an issue to discuss
  • s

    sathoro

    04/27/2023, 4:07 PM
    I stumbled on this same thing in our bundle the other day 🤣 yeah it is a list of TLDs
  • t

    Tom Sherman

    04/27/2023, 4:08 PM
    you could look at deno deploy if you have the opportunity to choose. just tested it and you can import this lib on deno with npm compat. i don't think they have startup time constraints
  • t

    Tom Sherman

    04/27/2023, 4:08 PM
    altho i guess we're talking about CPU time aren't we
  • w

    wesbos

    04/27/2023, 4:08 PM
    Im just doign exactly that haha
  • t

    Tom Sherman

    04/27/2023, 4:08 PM
    so same issue there
  • w

    wesbos

    04/27/2023, 4:08 PM
    I hav a few sites on Deno Deploy right now
  • w

    wesbos

    04/27/2023, 4:08 PM
    This is another worker I have, how am I doing 100ms here on a 10ms plan?
  • k

    kian

    04/27/2023, 4:09 PM
    rollover & grace periods
1...242224232424...2509Latest