New Architecture

[apteryxxyz]

Code

• Plugin class that Client and LinkedRoles extends
• createHandle method
• Node.js adapter
• Cloudflare adapter (added, untested)
  • Fix needed reference issue
• Bun adapter
• Next.js adapter
• create-carbon new templates
  • Customise Next.js default page.tsx
• Add ability to protect routes
• Add ability to disable routes
• Move linked roles setting metadata to route too
• Rewrite template readme to link to docs instead

Testing

• Node.js interactions
• Node.js linked roles
• Bun interactions
• Bun linked roles
• Next.js interactions (in Node.js)
• Next.js linked roles (in Node.js)
• Next.js interactions (on Vercel)
• Next.js linked roles (on Vercel)
• Cloudflare interactions
• Cloudflare linked roles
• Update example apps in apps/

Documentation

• Update documention

Closes #99

[changeset-bot]

🦋 Changeset detected

Latest commit: 0d754ea

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
create-carbon	Minor
@buape/carbon	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[thewilloftheshadow]

You can rebase on main to fix the formatting CI error

[thewilloftheshadow]

I love the direction this is going, its exciting!

Couple of changes and questions I have here so far, and I marked the TODOs as well.
Make sure everything that's marked as public has tsdoc on it even if they aren't generally made for the end user to use, because they'll show up in the generated docs site anyway. Might be good to use @internal on those too.

Also, there are a few lines that just end with // like there was a comment there that is now gone, is that intentional?

Good luck on your testing! 🎉

[apteryxxyz]

Regarding the // at the end of some lines, this is just me forcing some preferred formatting.

For example, Biome would format a piece of code like this:

await this.rest.put(Routes.applicationCommands(this.options.clientId), {
    body: commands
})

I myself don't like this, so I'll put // to force the formatting to this:

await this.rest.put(
    Routes.applicationCommands(this.options.clientId), //
    { body: commands }
)

[thewilloftheshadow]

Ah, that's a neat trick

[thewilloftheshadow]

Any idea on the docs api script failing? Haven't tried to fix it yet but I haven't dug into how this version works

[apteryxxyz]

Any idea on the docs api script failing? Haven't tried to fix it yet but I haven't dug into how this version works

Seems to be related to the copy-api.sh script, the action doesnt like escaped_name="${name//@/\\@}". As to why my best guess is the shell in the action isn't bash, referencing this Stack Overflow answer. I'm not really sure though

[apteryxxyz]

Any idea on the docs api script failing? Haven't tried to fix it yet but I haven't dug into how this version works

#185 fixes this issue

[apteryxxyz]

Finally done 😂😴