22 Commits
0.1.1 ... 0.3.0

Author SHA1 Message Date
MapleLeaf
e8c713bdf9 release v0.3.0 2022-01-11 16:41:01 -06:00
MapleLeaf
752402a177 add a link to instances guide from useInstance 2022-01-11 16:36:29 -06:00
MapleLeaf
c654b1ac3b upgrades 2022-01-11 16:35:23 -06:00
MapleLeaf
75ed66e1cd add useInstance 2022-01-11 16:28:41 -06:00
MapleLeaf
b3464fd05e ReacordTester: accept and pass initial content 2022-01-11 16:22:51 -06:00
MapleLeaf
86aa408897 ensure pruneNullishValues is called with plain objects
it would probably be better to add an actual check and error for this, but meh
2022-01-11 16:22:29 -06:00
MapleLeaf
42e8cbf754 pruneNullishValues: remove unneeded conditional 2022-01-11 16:21:54 -06:00
MapleLeaf
53331880ba update todo 2022-01-11 15:28:49 -06:00
MapleLeaf
bee2a83228 release v0.2.0 2022-01-11 00:47:23 -06:00
Darius
5b2db4dd69 Convert tests to Vitest (#4) 2022-01-11 00:33:13 -06:00
MapleLeaf
661a253d8c update todo 2022-01-10 17:15:50 -06:00
MapleLeaf
5b68a1f0ea yaml is bad 2022-01-10 17:07:24 -06:00
MapleLeaf
8c4a450bfb update paths on deploy workflow 2022-01-10 17:06:45 -06:00
MapleLeaf
14b74a0a3f remove readme in docs 2022-01-10 17:05:33 -06:00
MapleLeaf
f2d309b2a7 add eslint override for cypress 2022-01-10 16:52:31 -06:00
MapleLeaf
adeb0fb0d9 fix type checking for cypress 2022-01-10 16:04:09 -06:00
Sami Ibrahim
4f8308dc73 Update README.md (#5) 2022-01-10 15:33:29 -06:00
MapleLeaf
f5d8fa5271 docker: set node env after install 2022-01-10 15:11:38 -06:00
MapleLeaf
ac759412b3 lint fix 2022-01-10 15:10:50 -06:00
MapleLeaf
608b008659 fix typress type error 2022-01-10 15:10:23 -06:00
MapleLeaf
87cbf1b74d docs: add cypress test 2022-01-10 15:06:55 -06:00
MapleLeaf
4f463bfa23 link directly to alpine dist js
just to reduce network round trips
2022-01-10 14:44:19 -06:00
80 changed files with 1503 additions and 2065 deletions

View File

@@ -14,5 +14,13 @@
"rules": { "rules": {
"import/no-unused-modules": "off", "import/no-unused-modules": "off",
"unicorn/prevent-abbreviations": "off" "unicorn/prevent-abbreviations": "off"
},
"overrides": [
{
"files": ["packages/website/cypress/**"],
"parserOptions": {
"project": "./packages/website/cypress/tsconfig.json"
} }
}
]
} }

View File

@@ -1,9 +1,10 @@
name: deploy docs name: deploy website
on: on:
push: push:
branches: [main] branches: [main]
paths: [packages/docs/**, reacord/library/**/*.ts] paths:
- "packages/website/**"
- "reacord/library/**/*.{ts,tsx}"
jobs: jobs:
deploy: deploy:
runs-on: ubuntu-latest runs-on: ubuntu-latest

View File

@@ -11,20 +11,24 @@ env:
TEST_GUILD_ID: ${{ secrets.TEST_GUILD_ID }} TEST_GUILD_ID: ${{ secrets.TEST_GUILD_ID }}
jobs: jobs:
run-scripts: run-commands:
strategy: strategy:
matrix:
scripts:
- name: test
script: coverage
- name: lint
script: lint
- name: typecheck
script: typecheck
- name: build
script: build
fail-fast: false fail-fast: false
name: ${{ matrix.scripts.name }} matrix:
command:
# if these run in the same process, it dies,
# so we test them separate
- name: test reacord
run: pnpm test -C packages/reacord
- name: test website
run: pnpm test -C packages/website
- name: build
run: pnpm build --recursive
- name: lint
run: pnpm lint
- name: typecheck
run: pnpm typecheck --parallel
name: ${{ matrix.command.name }}
runs-on: ubuntu-latest runs-on: ubuntu-latest
steps: steps:
- uses: actions/checkout@v2 - uses: actions/checkout@v2
@@ -34,4 +38,4 @@ jobs:
node-version: "16" node-version: "16"
- run: npm i -g pnpm - run: npm i -g pnpm
- run: pnpm install --frozen-lockfile - run: pnpm install --frozen-lockfile
- run: pnpm run --recursive ${{ matrix.scripts.script }} - run: ${{ matrix.command.run }}

View File

@@ -1,5 +1,7 @@
FROM node:lts-slim FROM node:lts-slim
ENV CYPRESS_INSTALL_BINARY=0
WORKDIR /app WORKDIR /app
COPY / ./ COPY / ./
@@ -7,7 +9,7 @@ RUN ls -R
RUN npm install -g pnpm RUN npm install -g pnpm
RUN pnpm install --unsafe-perm --frozen-lockfile RUN pnpm install --unsafe-perm --frozen-lockfile
RUN pnpm run build -C packages/docs RUN pnpm run build -C packages/website
ENV NODE_ENV=production ENV NODE_ENV=production
CMD [ "pnpm", "-C", "packages/docs", "start" ] CMD [ "pnpm", "-C", "packages/website", "start" ]

View File

@@ -1,7 +1,23 @@
# reacord # reacord
Create interactive Discord messages using React!
Create interactive Discord messages using React! [Visit the docs to get started.](https://reacord.fly.dev/guides/getting-started) ## Installation
```console
# npm
npm install reacord react discord.js
# yarn
yarn add reacord react discord.js
# pnpm
pnpm add reacord react discord.js
```
## Get Started
[Visit the docs to get started.](https://reacord.fly.dev/guides/getting-started)
## Example
<!-- prettier-ignore --> <!-- prettier-ignore -->
```tsx ```tsx
import * as React from "react" import * as React from "react"

View File

@@ -9,8 +9,8 @@
"@itsmapleleaf/configs": "^1.1.2" "@itsmapleleaf/configs": "^1.1.2"
}, },
"devDependencies": { "devDependencies": {
"@typescript-eslint/eslint-plugin": "^5.9.0", "@typescript-eslint/eslint-plugin": "^5.9.1",
"@typescript-eslint/parser": "^5.9.0", "@typescript-eslint/parser": "^5.9.1",
"eslint": "^8.6.0", "eslint": "^8.6.0",
"eslint-config-prettier": "^8.3.0", "eslint-config-prettier": "^8.3.0",
"eslint-import-resolver-typescript": "^2.5.0", "eslint-import-resolver-typescript": "^2.5.0",
@@ -18,7 +18,7 @@
"eslint-plugin-jsx-a11y": "^6.5.1", "eslint-plugin-jsx-a11y": "^6.5.1",
"eslint-plugin-react": "^7.28.0", "eslint-plugin-react": "^7.28.0",
"eslint-plugin-react-hooks": "^4.3.0", "eslint-plugin-react-hooks": "^4.3.0",
"eslint-plugin-unicorn": "^39.0.0", "eslint-plugin-unicorn": "^40.0.0",
"prettier": "^2.5.1", "prettier": "^2.5.1",
"typescript": "^4.5.4" "typescript": "^4.5.4"
}, },

View File

@@ -1,53 +0,0 @@
# Welcome to Remix!
- [Remix Docs](https://remix.run/docs)
## Development
From your terminal:
```sh
npm run dev
```
This starts your app in development mode, rebuilding assets on file changes.
## Deployment
First, build your app for production:
```sh
npm run build
```
Then run the app in production mode:
```sh
npm start
```
Now you'll need to pick a host to deploy it to.
### DIY
If you're familiar with deploying node applications, the built-in Remix app server is production-ready.
Make sure to deploy the output of `remix build`
- `build/`
- `public/build/`
### Using a Template
When you ran `npx create-remix@latest` there were a few choices for hosting. You can run that again to create a new project, then copy over your `app/` folder to the new project that's pre-configured for your target server.
```sh
cd ..
# create a new project, and pick a pre-configured host
npx create-remix@latest
cd my-new-remix-app
# remove the new project's app (not the old one!)
rm -rf app
# copy your app over
cp -R ../my-old-remix-app/app app
```

View File

@@ -2,6 +2,7 @@ import type {
CamelCasedPropertiesDeep, CamelCasedPropertiesDeep,
SnakeCasedPropertiesDeep, SnakeCasedPropertiesDeep,
} from "type-fest" } from "type-fest"
import { expect, test } from "vitest"
import { camelCaseDeep, snakeCaseDeep } from "./convert-object-property-case" import { camelCaseDeep, snakeCaseDeep } from "./convert-object-property-case"
test("camelCaseDeep", () => { test("camelCaseDeep", () => {
@@ -12,12 +13,14 @@ test("camelCaseDeep", () => {
someOtherProp: "someOtherValue", someOtherProp: "someOtherValue",
} }
expect(camelCaseDeep(input)).toEqual<CamelCasedPropertiesDeep<typeof input>>({ const expected: CamelCasedPropertiesDeep<typeof input> = {
someProp: { someProp: {
someDeepProp: "some_deep_value", someDeepProp: "some_deep_value",
}, },
someOtherProp: "someOtherValue", someOtherProp: "someOtherValue",
}) }
expect(camelCaseDeep(input)).toEqual(expected)
}) })
test("snakeCaseDeep", () => { test("snakeCaseDeep", () => {
@@ -28,10 +31,12 @@ test("snakeCaseDeep", () => {
some_other_prop: "someOtherValue", some_other_prop: "someOtherValue",
} }
expect(snakeCaseDeep(input)).toEqual<SnakeCasedPropertiesDeep<typeof input>>({ const expected: SnakeCasedPropertiesDeep<typeof input> = {
some_prop: { some_prop: {
some_deep_prop: "someDeepValue", some_deep_prop: "someDeepValue",
}, },
some_other_prop: "someOtherValue", some_other_prop: "someOtherValue",
}) }
expect(snakeCaseDeep(input)).toEqual(expected)
}) })

View File

@@ -0,0 +1,7 @@
export function isObject<T>(
value: T,
): value is Exclude<T, Primitive | AnyFunction> {
return typeof value === "object" && value !== null
}
type Primitive = string | number | boolean | undefined | null
type AnyFunction = (...args: any[]) => any

View File

@@ -1,15 +0,0 @@
export function pruneNullishValues<T extends object>(
object: T,
): PruneNullishValues<T> {
const result: any = {}
for (const [key, value] of Object.entries(object)) {
if (value != undefined) {
result[key] = value
}
}
return result
}
type PruneNullishValues<T> = {
[Key in keyof T]: NonNullable<T[Key]>
}

View File

@@ -0,0 +1,27 @@
import { isObject } from "./is-object"
export function pruneNullishValues<T>(input: T): PruneNullishValues<T> {
if (Array.isArray(input)) {
return input.filter(Boolean).map((item) => pruneNullishValues(item)) as any
}
if (!isObject(input)) {
return input as any
}
const result: any = {}
for (const [key, value] of Object.entries(input)) {
if (value != undefined) {
result[key] = pruneNullishValues(value)
}
}
return result
}
type PruneNullishValues<Input> = Input extends ReadonlyArray<infer Value>
? ReadonlyArray<NonNullable<Value>>
: Input extends object
? {
[Key in keyof Input]: NonNullable<Input[Key]>
}
: Input

View File

@@ -6,5 +6,5 @@ export async function rejectAfter(
error: unknown = `rejected after ${timeMs}ms`, error: unknown = `rejected after ${timeMs}ms`,
): Promise<never> { ): Promise<never> {
await setTimeout(timeMs) await setTimeout(timeMs)
return Promise.reject(toError(error)) throw toError(error)
} }

View File

@@ -1,15 +0,0 @@
/** @type {import('@jest/types').Config.InitialOptions} */
const config = {
transform: {
"^.+\\.[jt]sx?$": ["esbuild-jest", { format: "esm", sourcemap: true }],
},
extensionsToTreatAsEsm: [".ts", ".tsx"],
moduleNameMapper: {
"^(\\.\\.?/.+)\\.jsx?$": "$1",
},
verbose: true,
cache: false,
coverageReporters: ["text", "text-summary", "html"],
coveragePathIgnorePatterns: ["discord-js-adapter", "test/setup-testing"],
}
export default config

View File

@@ -4,7 +4,11 @@ import React from "react"
import { isInstanceOf } from "../../../helpers/is-instance-of" import { isInstanceOf } from "../../../helpers/is-instance-of"
import { ReacordElement } from "../../internal/element.js" import { ReacordElement } from "../../internal/element.js"
import type { ComponentInteraction } from "../../internal/interaction" import type { ComponentInteraction } from "../../internal/interaction"
import type { ActionRow, MessageOptions } from "../../internal/message" import type {
ActionRow,
ActionRowItem,
MessageOptions,
} from "../../internal/message"
import { Node } from "../../internal/node.js" import { Node } from "../../internal/node.js"
import type { ComponentEvent } from "../component-event" import type { ComponentEvent } from "../component-event"
import { OptionNode } from "./option-node" import { OptionNode } from "./option-node"
@@ -108,19 +112,25 @@ class SelectNode extends Node<SelectProps> {
...props ...props
} = this.props } = this.props
actionRow.push({ const item: ActionRowItem = {
...props, ...props,
type: "select", type: "select",
customId: this.customId, customId: this.customId,
options, options,
// I'm not counting on people using value and values at the same time, values: [],
// but maybe we should resolve this differently anyhow? e.g. one should override the other }
// or just warn if there are both
// or... try some other alternative design entirely if (multiple) {
values: [...(values || []), ...(value ? [value] : [])], item.minValues = minValues
minValues: multiple ? minValues : undefined, item.maxValues = maxValues
maxValues: multiple ? Math.max(minValues, maxValues) : undefined, if (values) item.values = values
}) }
if (!multiple && value != undefined) {
item.values = [value]
}
actionRow.push(item)
} }
override handleComponentInteraction( override handleComponentInteraction(

View File

@@ -0,0 +1,20 @@
import * as React from "react"
import { raise } from "../../helpers/raise"
import type { ReacordInstance } from "./instance"
const Context = React.createContext<ReacordInstance | undefined>(undefined)
export const InstanceProvider = Context.Provider
/**
* Get the associated instance for the current component.
*
* @category Core
* @see https://reacord.fly.dev/guides/use-instance
*/
export function useInstance(): ReacordInstance {
return (
React.useContext(Context) ??
raise("Could not find instance, was this component rendered via Reacord?")
)
}

View File

@@ -3,7 +3,7 @@ import * as Discord from "discord.js"
import type { ReactNode } from "react" import type { ReactNode } from "react"
import type { Except } from "type-fest" import type { Except } from "type-fest"
import { pick } from "../../helpers/pick" import { pick } from "../../helpers/pick"
import { pruneNullishValues } from "../../helpers/prune-null-values" import { pruneNullishValues } from "../../helpers/prune-nullish-values"
import { raise } from "../../helpers/raise" import { raise } from "../../helpers/raise"
import { toUpper } from "../../helpers/to-upper" import { toUpper } from "../../helpers/to-upper"
import type { ComponentInteraction } from "../internal/interaction" import type { ComponentInteraction } from "../internal/interaction"
@@ -154,7 +154,8 @@ export class ReacordDiscordJs extends Reacord {
// todo please dear god clean this up // todo please dear god clean this up
const channel: ChannelInfo = interaction.channel const channel: ChannelInfo = interaction.channel
? { ? {
...pick(pruneNullishValues(interaction.channel), [ ...pruneNullishValues(
pick(interaction.channel, [
"topic", "topic",
"nsfw", "nsfw",
"lastMessageId", "lastMessageId",
@@ -162,6 +163,7 @@ export class ReacordDiscordJs extends Reacord {
"parentId", "parentId",
"rateLimitPerUser", "rateLimitPerUser",
]), ]),
),
id: interaction.channelId, id: interaction.channelId,
} }
: raise("Non-channel interactions are not supported") : raise("Non-channel interactions are not supported")
@@ -190,7 +192,8 @@ export class ReacordDiscordJs extends Reacord {
const member: GuildMemberInfo | undefined = const member: GuildMemberInfo | undefined =
interaction.member instanceof Discord.GuildMember interaction.member instanceof Discord.GuildMember
? { ? {
...pick(pruneNullishValues(interaction.member), [ ...pruneNullishValues(
pick(interaction.member, [
"id", "id",
"nick", "nick",
"displayName", "displayName",
@@ -199,6 +202,7 @@ export class ReacordDiscordJs extends Reacord {
"color", "color",
"pending", "pending",
]), ]),
),
displayName: interaction.member.displayName, displayName: interaction.member.displayName,
roles: [...interaction.member.roles.cache.map((role) => role.id)], roles: [...interaction.member.roles.cache.map((role) => role.id)],
joinedAt: interaction.member.joinedAt?.toISOString(), joinedAt: interaction.member.joinedAt?.toISOString(),
@@ -210,18 +214,15 @@ export class ReacordDiscordJs extends Reacord {
const guild: GuildInfo | undefined = interaction.guild const guild: GuildInfo | undefined = interaction.guild
? { ? {
...pick(pruneNullishValues(interaction.guild), ["id", "name"]), ...pruneNullishValues(pick(interaction.guild, ["id", "name"])),
member: member ?? raise("unexpected: member is undefined"), member: member ?? raise("unexpected: member is undefined"),
} }
: undefined : undefined
const user: UserInfo = { const user: UserInfo = {
...pick(pruneNullishValues(interaction.user), [ ...pruneNullishValues(
"id", pick(interaction.user, ["id", "username", "discriminator", "tag"]),
"username", ),
"discriminator",
"tag",
]),
avatarUrl: interaction.user.avatarURL()!, avatarUrl: interaction.user.avatarURL()!,
accentColor: interaction.user.accentColor ?? undefined, accentColor: interaction.user.accentColor ?? undefined,
} }

View File

@@ -1,8 +1,10 @@
import type { ReactNode } from "react" import type { ReactNode } from "react"
import React from "react"
import type { ComponentInteraction } from "../internal/interaction" import type { ComponentInteraction } from "../internal/interaction"
import { reconciler } from "../internal/reconciler.js" import { reconciler } from "../internal/reconciler.js"
import type { Renderer } from "../internal/renderers/renderer" import type { Renderer } from "../internal/renderers/renderer"
import type { ReacordInstance } from "./instance" import type { ReacordInstance } from "./instance"
import { InstanceProvider } from "./instance-context"
/** /**
* @category Core * @category Core
@@ -47,13 +49,12 @@ export abstract class Reacord {
const container = reconciler.createContainer(renderer, 0, false, {}) const container = reconciler.createContainer(renderer, 0, false, {})
if (initialContent !== undefined) { const instance: ReacordInstance = {
reconciler.updateContainer(initialContent, container)
}
return {
render: (content: ReactNode) => { render: (content: ReactNode) => {
reconciler.updateContainer(content, container) reconciler.updateContainer(
<InstanceProvider value={instance}>{content}</InstanceProvider>,
container,
)
}, },
deactivate: () => { deactivate: () => {
this.deactivate(renderer) this.deactivate(renderer)
@@ -63,6 +64,12 @@ export abstract class Reacord {
renderer.destroy() renderer.destroy()
}, },
} }
if (initialContent !== undefined) {
instance.render(initialContent)
}
return instance
} }
private deactivate(renderer: Renderer) { private deactivate(renderer: Renderer) {

View File

@@ -9,9 +9,12 @@ export type MessageOptions = {
actionRows: ActionRow[] actionRows: ActionRow[]
} }
export type ActionRow = Array< export type ActionRow = ActionRowItem[]
MessageButtonOptions | MessageLinkOptions | MessageSelectOptions
> export type ActionRowItem =
| MessageButtonOptions
| MessageLinkOptions
| MessageSelectOptions
export type MessageButtonOptions = { export type MessageButtonOptions = {
type: "button" type: "button"

View File

@@ -13,6 +13,6 @@ export * from "./core/components/link"
export * from "./core/components/option" export * from "./core/components/option"
export * from "./core/components/select" export * from "./core/components/select"
export * from "./core/instance" export * from "./core/instance"
export { useInstance } from "./core/instance-context"
export * from "./core/reacord" export * from "./core/reacord"
export * from "./core/reacord-discord-js" export * from "./core/reacord-discord-js"
export * from "./core/reacord-tester"

View File

@@ -2,7 +2,7 @@
"name": "reacord", "name": "reacord",
"type": "module", "type": "module",
"description": "Create interactive Discord messages using React.", "description": "Create interactive Discord messages using React.",
"version": "0.1.1", "version": "0.3.0",
"types": "./dist/main.d.ts", "types": "./dist/main.d.ts",
"files": [ "files": [
"dist" "dist"
@@ -20,20 +20,19 @@
"scripts": { "scripts": {
"build": "tsup-node library/main.ts --target node16 --format cjs,esm --dts --sourcemap", "build": "tsup-node library/main.ts --target node16 --format cjs,esm --dts --sourcemap",
"build-watch": "pnpm build -- --watch", "build-watch": "pnpm build -- --watch",
"test": "node --experimental-vm-modules --no-warnings ./node_modules/jest/bin/jest.js --colors", "test": "vitest --coverage --no-watch",
"test-watch": "pnpm test -- --watch", "test-dev": "vitest",
"coverage": "pnpm test -- --coverage",
"typecheck": "tsc --noEmit", "typecheck": "tsc --noEmit",
"playground": "nodemon --exec esmo --ext ts,tsx ./playground/main.tsx", "playground": "nodemon --exec esmo --ext ts,tsx --inspect=5858 --enable-source-maps ./playground/main.tsx",
"release": "release-it" "release": "release-it"
}, },
"dependencies": { "dependencies": {
"@types/node": "*", "@types/node": "*",
"@types/react": "*", "@types/react": "*",
"@types/react-reconciler": "^0.26.4", "@types/react-reconciler": "^0.26.4",
"nanoid": "^3.1.30", "nanoid": "^3.1.31",
"react-reconciler": "^0.26.2", "react-reconciler": "^0.26.2",
"rxjs": "^7.5.1" "rxjs": "^7.5.2"
}, },
"peerDependencies": { "peerDependencies": {
"discord.js": "^13.3", "discord.js": "^13.3",
@@ -45,16 +44,13 @@
} }
}, },
"devDependencies": { "devDependencies": {
"@jest/globals": "^27.4.6",
"@types/jest": "^27.4.0",
"@types/lodash-es": "^4.17.5", "@types/lodash-es": "^4.17.5",
"c8": "^7.11.0", "c8": "^7.11.0",
"discord.js": "^13.5.1", "discord.js": "^13.5.1",
"dotenv": "^10.0.0", "dotenv": "^11.0.0",
"esbuild": "latest", "esbuild": "latest",
"esbuild-jest": "^0.5.0", "esbuild-jest": "^0.5.0",
"esmo": "^0.13.0", "esmo": "^0.13.0",
"jest": "^27.4.7",
"lodash-es": "^4.17.21", "lodash-es": "^4.17.21",
"nodemon": "^2.0.15", "nodemon": "^2.0.15",
"prettier": "^2.5.1", "prettier": "^2.5.1",
@@ -63,7 +59,9 @@
"release-it": "^14.12.1", "release-it": "^14.12.1",
"tsup": "^5.11.11", "tsup": "^5.11.11",
"type-fest": "^2.9.0", "type-fest": "^2.9.0",
"typescript": "^4.5.4" "typescript": "^4.5.4",
"vite": "^2.7.10",
"vitest": "^0.0.141"
}, },
"resolutions": { "resolutions": {
"esbuild": "latest" "esbuild": "latest"

View File

@@ -1,7 +1,7 @@
import { Client } from "discord.js" import { Client } from "discord.js"
import "dotenv/config" import "dotenv/config"
import React from "react" import React from "react"
import { Button, ReacordDiscordJs } from "../library/main" import { Button, ReacordDiscordJs, useInstance } from "../library/main"
import { createCommandHandler } from "./command-handler" import { createCommandHandler } from "./command-handler"
import { Counter } from "./counter" import { Counter } from "./counter"
import { FruitSelect } from "./fruit-select" import { FruitSelect } from "./fruit-select"
@@ -93,6 +93,17 @@ createCommandHandler(client, [
) )
}, },
}, },
{
name: "delete-this",
description: "delete this",
run: (interaction) => {
function DeleteThis() {
const instance = useInstance()
return <Button label="delete this" onClick={() => instance.destroy()} />
}
reacord.reply(interaction, <DeleteThis />)
},
},
]) ])
await client.login(process.env.TEST_BOT_TOKEN) await client.login(process.env.TEST_BOT_TOKEN)

View File

@@ -1,6 +1,7 @@
import React from "react" import React from "react"
import { ReacordTester } from "../library/core/reacord-tester" import { test } from "vitest"
import { ActionRow, Button, Select } from "../library/main" import { ActionRow, Button, Select } from "../library/main"
import { ReacordTester } from "./test-adapter"
const testing = new ReacordTester() const testing = new ReacordTester()

View File

@@ -1,2 +1,3 @@
import { test } from "vitest"
test.todo("channel message renderer") test.todo("channel message renderer")
export {}

View File

@@ -1,2 +1,3 @@
import { test } from "vitest"
test.todo("discord js integration") test.todo("discord js integration")
export {}

View File

@@ -1,5 +1,5 @@
import React from "react" import React from "react"
import { ReacordTester } from "../library/core/reacord-tester" import { test } from "vitest"
import { import {
Embed, Embed,
EmbedAuthor, EmbedAuthor,
@@ -9,6 +9,7 @@ import {
EmbedThumbnail, EmbedThumbnail,
EmbedTitle, EmbedTitle,
} from "../library/main" } from "../library/main"
import { ReacordTester } from "./test-adapter"
const testing = new ReacordTester() const testing = new ReacordTester()

View File

@@ -1,2 +1,2 @@
import { test } from "vitest"
test.todo("ephemeral reply") test.todo("ephemeral reply")
export {}

View File

@@ -1,3 +1,4 @@
import { test } from "vitest"
test.todo("button onClick") test.todo("button onClick")
test.todo("select onChange") test.todo("select onChange")
export {}

View File

@@ -1,6 +1,7 @@
import React from "react" import React from "react"
import { ReacordTester } from "../library/core/reacord-tester" import { test } from "vitest"
import { Link } from "../library/main" import { Link } from "../library/main"
import { ReacordTester } from "./test-adapter"
const tester = new ReacordTester() const tester = new ReacordTester()

View File

@@ -1,11 +1,7 @@
import * as React from "react" import * as React from "react"
import { import { test } from "vitest"
Button, import { Button, Embed, EmbedField, EmbedTitle } from "../library/main"
Embed, import { ReacordTester } from "./test-adapter"
EmbedField,
EmbedTitle,
ReacordTester,
} from "../library/main"
test("rendering behavior", async () => { test("rendering behavior", async () => {
const tester = new ReacordTester() const tester = new ReacordTester()

View File

@@ -1,6 +1,6 @@
import { test } from "vitest"
// test that the interaction update is _eventually_ deferred if there's no component update, // test that the interaction update is _eventually_ deferred if there's no component update,
// and that update isn't called after the fact // and that update isn't called after the fact
// ...somehow // ...somehow
test.todo("defer update timeout") test.todo("defer update timeout")
export {}

View File

@@ -1,10 +1,11 @@
import { jest } from "@jest/globals"
import React, { useState } from "react" import React, { useState } from "react"
import { Button, Option, ReacordTester, Select } from "../library/main" import { expect, fn, test } from "vitest"
import { Button, Option, Select } from "../library/main"
import { ReacordTester } from "./test-adapter"
test("single select", async () => { test("single select", async () => {
const tester = new ReacordTester() const tester = new ReacordTester()
const onSelect = jest.fn() const onSelect = fn()
function TestSelect() { function TestSelect() {
const [value, setValue] = useState<string>() const [value, setValue] = useState<string>()
@@ -74,7 +75,7 @@ test("single select", async () => {
test("multiple select", async () => { test("multiple select", async () => {
const tester = new ReacordTester() const tester = new ReacordTester()
const onSelect = jest.fn() const onSelect = fn()
function TestSelect() { function TestSelect() {
const [values, setValues] = useState<string[]>([]) const [values, setValues] = useState<string[]>([])
@@ -125,13 +126,13 @@ test("multiple select", async () => {
expect(onSelect).toHaveBeenCalledTimes(0) expect(onSelect).toHaveBeenCalledTimes(0)
tester.findSelectByPlaceholder("select").select("1", "3") tester.findSelectByPlaceholder("select").select("1", "3")
await assertSelect(expect.arrayContaining(["1", "3"])) await assertSelect(expect.arrayContaining(["1", "3"]) as unknown as string[])
expect(onSelect).toHaveBeenCalledWith( expect(onSelect).toHaveBeenCalledWith(
expect.objectContaining({ values: expect.arrayContaining(["1", "3"]) }), expect.objectContaining({ values: expect.arrayContaining(["1", "3"]) }),
) )
tester.findSelectByPlaceholder("select").select("2") tester.findSelectByPlaceholder("select").select("2")
await assertSelect(expect.arrayContaining(["2"])) await assertSelect(expect.arrayContaining(["2"]) as unknown as string[])
expect(onSelect).toHaveBeenCalledWith( expect(onSelect).toHaveBeenCalledWith(
expect.objectContaining({ values: expect.arrayContaining(["2"]) }), expect.objectContaining({ values: expect.arrayContaining(["2"]) }),
) )

View File

@@ -4,37 +4,41 @@ import { nanoid } from "nanoid"
import { nextTick } from "node:process" import { nextTick } from "node:process"
import { promisify } from "node:util" import { promisify } from "node:util"
import type { ReactNode } from "react" import type { ReactNode } from "react"
import { logPretty } from "../../helpers/log-pretty" import { expect } from "vitest"
import { omit } from "../../helpers/omit" import { logPretty } from "../helpers/log-pretty"
import { raise } from "../../helpers/raise" import { omit } from "../helpers/omit"
import type { Channel } from "../internal/channel" import { pruneNullishValues } from "../helpers/prune-nullish-values"
import { Container } from "../internal/container" import { raise } from "../helpers/raise"
import type {
ButtonInteraction,
CommandInteraction,
SelectInteraction,
} from "../internal/interaction"
import type {
Message,
MessageButtonOptions,
MessageOptions,
MessageSelectOptions,
} from "../internal/message"
import { ChannelMessageRenderer } from "../internal/renderers/channel-message-renderer"
import { InteractionReplyRenderer } from "../internal/renderers/interaction-reply-renderer"
import type { import type {
ChannelInfo, ChannelInfo,
GuildInfo, GuildInfo,
MessageInfo, MessageInfo,
UserInfo, UserInfo,
} from "./component-event" } from "../library/core/component-event"
import type { ButtonClickEvent } from "./components/button" import type { ButtonClickEvent } from "../library/core/components/button"
import type { SelectChangeEvent } from "./components/select" import type { SelectChangeEvent } from "../library/core/components/select"
import type { ReacordInstance } from "./instance" import type { ReacordInstance } from "../library/core/instance"
import { Reacord } from "./reacord" import { Reacord } from "../library/core/reacord"
import type { Channel } from "../library/internal/channel"
import { Container } from "../library/internal/container"
import type {
ButtonInteraction,
CommandInteraction,
SelectInteraction,
} from "../library/internal/interaction"
import type {
Message,
MessageButtonOptions,
MessageOptions,
MessageSelectOptions,
} from "../library/internal/message"
import { ChannelMessageRenderer } from "../library/internal/renderers/channel-message-renderer"
import { InteractionReplyRenderer } from "../library/internal/renderers/interaction-reply-renderer"
const nextTickPromise = promisify(nextTick) const nextTickPromise = promisify(nextTick)
export type MessageSample = ReturnType<ReacordTester["sampleMessages"]>[0]
/** /**
* A Record adapter for automated tests. WIP * A Record adapter for automated tests. WIP
*/ */
@@ -49,33 +53,32 @@ export class ReacordTester extends Reacord {
return [...this.messageContainer] return [...this.messageContainer]
} }
override send(): ReacordInstance { override send(initialContent?: ReactNode): ReacordInstance {
return this.createInstance( return this.createInstance(
new ChannelMessageRenderer(new TestChannel(this.messageContainer)), new ChannelMessageRenderer(new TestChannel(this.messageContainer)),
initialContent,
) )
} }
override reply(): ReacordInstance { override reply(initialContent?: ReactNode): ReacordInstance {
return this.createInstance( return this.createInstance(
new InteractionReplyRenderer( new InteractionReplyRenderer(
new TestCommandInteraction(this.messageContainer), new TestCommandInteraction(this.messageContainer),
), ),
initialContent,
) )
} }
override ephemeralReply(): ReacordInstance { override ephemeralReply(initialContent?: ReactNode): ReacordInstance {
return this.reply() return this.reply(initialContent)
} }
async assertMessages(expected: ReturnType<this["sampleMessages"]>) { async assertMessages(expected: MessageSample[]) {
await nextTickPromise() await nextTickPromise()
expect(this.sampleMessages()).toEqual(expected) expect(this.sampleMessages()).toEqual(expected)
} }
async assertRender( async assertRender(content: ReactNode, expected: MessageSample[]) {
content: ReactNode,
expected: ReturnType<this["sampleMessages"]>,
) {
const instance = this.reply() const instance = this.reply()
instance.render(content) instance.render(content)
await this.assertMessages(expected) await this.assertMessages(expected)
@@ -87,14 +90,21 @@ export class ReacordTester extends Reacord {
} }
sampleMessages() { sampleMessages() {
return this.messages.map((message) => ({ return pruneNullishValues(
this.messages.map((message) => ({
...message.options, ...message.options,
actionRows: message.options.actionRows.map((row) => actionRows: message.options.actionRows.map((row) =>
row.map((component) => row.map((component) =>
omit(component, ["customId", "onClick", "onSelect", "onSelectValue"]), omit(component, [
"customId",
"onClick",
"onSelect",
"onSelectValue",
]),
), ),
), ),
})) })),
)
} }
findButtonByLabel(label: string) { findButtonByLabel(label: string) {
@@ -265,11 +275,11 @@ class TestComponentEvent {
guild: GuildInfo = {} as any // todo guild: GuildInfo = {} as any // todo
reply(content?: ReactNode): ReacordInstance { reply(content?: ReactNode): ReacordInstance {
return this.tester.reply() return this.tester.reply(content)
} }
ephemeralReply(content?: ReactNode): ReacordInstance { ephemeralReply(content?: ReactNode): ReacordInstance {
return this.tester.ephemeralReply() return this.tester.ephemeralReply(content)
} }
} }

View File

@@ -0,0 +1,72 @@
import React from "react"
import { describe, expect, it } from "vitest"
import type { ReacordInstance } from "../library/main"
import { Button, useInstance } from "../library/main"
import type { MessageSample } from "./test-adapter"
import { ReacordTester } from "./test-adapter"
describe("useInstance", () => {
it("returns the instance of itself", async () => {
let instanceFromHook: ReacordInstance | undefined
function TestComponent({ name }: { name: string }) {
const instance = useInstance()
instanceFromHook ??= instance
return (
<>
<Button
label={`create ${name}`}
onClick={(event) => {
event.reply(<TestComponent name="child" />)
}}
/>
<Button
label={`destroy ${name}`}
onClick={() => instance.destroy()}
/>
</>
)
}
function messageOutput(name: string): MessageSample {
return {
content: "",
embeds: [],
actionRows: [
[
{
type: "button",
label: `create ${name}`,
style: "secondary",
},
{
type: "button",
label: `destroy ${name}`,
style: "secondary",
},
],
],
}
}
const tester = new ReacordTester()
const instance = tester.send(<TestComponent name="parent" />)
await tester.assertMessages([messageOutput("parent")])
expect(instanceFromHook).toBe(instance)
tester.findButtonByLabel("create parent").click()
await tester.assertMessages([
messageOutput("parent"),
messageOutput("child"),
])
// this test ensures that the only the child instance is destroyed,
// and not the parent instance
tester.findButtonByLabel("destroy child").click()
await tester.assertMessages([messageOutput("parent")])
tester.findButtonByLabel("destroy parent").click()
await tester.assertMessages([])
})
})

View File

@@ -0,0 +1,8 @@
/// <reference types="vitest" />
import { defineConfig } from "vite"
export default defineConfig({
build: {
sourcemap: true,
},
})

View File

@@ -5,3 +5,5 @@ node_modules
/public/build /public/build
.env .env
/public/api /public/api
cypress/videos
cypress/screenshots

View File

@@ -16,11 +16,12 @@ export function MainNavigation() {
<AppLink {...link} key={link.to} className={linkClass} /> <AppLink {...link} key={link.to} className={linkClass} />
))} ))}
</div> </div>
<div className="md:hidden" id="main-navigation-popover"> <div className="md:hidden">
<PopoverMenu> <PopoverMenu>
{mainLinks.map((link) => ( {mainLinks.map((link) => (
<AppLink <AppLink
{...link} {...link}
role="menuitem"
key={link.to} key={link.to}
className={PopoverMenu.itemClass} className={PopoverMenu.itemClass}
/> />
@@ -29,6 +30,7 @@ export function MainNavigation() {
{guideLinks.map(({ link }) => ( {guideLinks.map(({ link }) => (
<AppLink <AppLink
{...link} {...link}
role="menuitem"
key={link.to} key={link.to}
className={PopoverMenu.itemClass} className={PopoverMenu.itemClass}
/> />

View File

@@ -10,6 +10,7 @@ export function PopoverMenu({ children }: { children: React.ReactNode }) {
<MenuAlt4Icon className="w-6" /> <MenuAlt4Icon className="w-6" />
</button> </button>
<div <div
role="menu"
className={` className={`
w-48 max-h-[calc(100vh-4rem)] w-48 max-h-[calc(100vh-4rem)]
absolute right-0 top-[calc(100%+8px)] absolute right-0 top-[calc(100%+8px)]

View File

@@ -54,7 +54,7 @@ export default function App() {
<meta name="viewport" content="width=device-width,initial-scale=1" /> <meta name="viewport" content="width=device-width,initial-scale=1" />
<Meta /> <Meta />
<Links /> <Links />
<script defer src="//unpkg.com/alpinejs@3.7.1" /> <script defer src="https://unpkg.com/alpinejs@3.7.1/dist/cdn.min.js" />
</head> </head>
<body> <body>
<GuideLinksProvider value={data.guideLinks}> <GuideLinksProvider value={data.guideLinks}>

View File

@@ -0,0 +1,26 @@
---
meta:
title: useInstance
description: Using useInstance to get the current instance within a component
---
# useInstance
You can use `useInstance` to get the current [instance](/guides/sending-messages) within a component. This can be used to let a component destroy or deactivate itself.
```jsx
import { Button, useInstance } from "reacord"
function SelfDestruct() {
const instance = useInstance()
return (
<Button
style="danger"
label="delete this"
onClick={() => instance.destroy()}
/>
)
}
reacord.send(channelId, <SelfDestruct />)
```

View File

@@ -0,0 +1,3 @@
{
"baseUrl": "http://localhost:3000/"
}

View File

@@ -0,0 +1,5 @@
{
"name": "Using fixtures to represent data",
"email": "hello@cypress.io",
"body": "Fixtures are a great way to mock data for responses to routes"
}

View File

@@ -0,0 +1,12 @@
export {}
describe("main popover menu", () => {
it("should toggle on button click", () => {
cy.viewport(480, 720)
cy.visit("/")
cy.findByRole("button", { name: "Menu" }).click()
cy.findByRole("menu").should("be.visible")
cy.findByRole("button", { name: "Menu" }).click()
cy.findByRole("menu").should("not.exist")
})
})

View File

@@ -0,0 +1,22 @@
/// <reference types="cypress" />
// ***********************************************************
// This example plugins/index.js can be used to load plugins
//
// You can change the location of this file or turn off loading
// the plugins file with the 'pluginsFile' configuration option.
//
// You can read more here:
// https://on.cypress.io/plugins-guide
// ***********************************************************
// This function is called when a project is opened or re-opened (e.g. due to
// the project's config changing)
/**
* @type {Cypress.PluginConfig}
*/
// eslint-disable-next-line no-unused-vars
export default function cypressConfig(on, config) {
// `on` is used to hook into various events Cypress emits
// `config` is the resolved Cypress config
}

View File

@@ -0,0 +1,26 @@
// ***********************************************
// This example commands.js shows you how to
// create various custom commands and overwrite
// existing commands.
//
// For more comprehensive examples of custom
// commands please read more here:
// https://on.cypress.io/custom-commands
// ***********************************************
//
//
// -- This is a parent command --
// Cypress.Commands.add('login', (email, password) => { ... })
//
//
// -- This is a child command --
// Cypress.Commands.add('drag', { prevSubject: 'element'}, (subject, options) => { ... })
//
//
// -- This is a dual command --
// Cypress.Commands.add('dismiss', { prevSubject: 'optional'}, (subject, options) => { ... })
//
//
// -- This will overwrite an existing command --
// Cypress.Commands.overwrite('visit', (originalFn, url, options) => { ... })
import "@testing-library/cypress/add-commands"

View File

@@ -0,0 +1,20 @@
// ***********************************************************
// This example support/index.js is processed and
// loaded automatically before your test files.
//
// This is a great place to put global configuration and
// behavior that modifies Cypress.
//
// You can change the location of this file or turn off
// automatically serving support files with the
// 'supportFile' configuration option.
//
// You can read more here:
// https://on.cypress.io/configuration
// ***********************************************************
// Import commands.js using ES2015 syntax:
import './commands'
// Alternatively you can use CommonJS syntax:
// require('./commands')

View File

@@ -0,0 +1,8 @@
{
"extends": "../tsconfig.json",
"compilerOptions": {
"types": ["cypress", "@testing-library/cypress"]
},
"include": ["."],
"exclude": []
}

View File

@@ -1,12 +1,14 @@
{ {
"private": true, "private": true,
"name": "reacord-docs-new", "name": "website",
"scripts": { "scripts": {
"prepare": "remix setup node", "prepare": "remix setup node",
"dev": "concurrently 'typedoc --watch' 'remix dev'", "dev": "concurrently 'typedoc --watch' 'remix dev'",
"test": "node ./scripts/test.js",
"test-dev": "pnpm dev & wait-on http-get://localhost:3000 && cypress open",
"build": "typedoc && remix build", "build": "typedoc && remix build",
"start": "remix-serve build", "start": "remix-serve build",
"typecheck": "tsc --noEmit" "typecheck": "tsc --noEmit && tsc --project cypress/tsconfig.json --noEmit"
}, },
"dependencies": { "dependencies": {
"@heroicons/react": "^1.0.5", "@heroicons/react": "^1.0.5",
@@ -23,19 +25,24 @@
"react-dom": "^17.0.2", "react-dom": "^17.0.2",
"remix": "^1.1.1", "remix": "^1.1.1",
"remix-tailwind": "^0.2.1", "remix-tailwind": "^0.2.1",
"tailwindcss": "^3.0.12" "tailwindcss": "^3.0.13"
}, },
"devDependencies": { "devDependencies": {
"@remix-run/dev": "^1.1.1", "@remix-run/dev": "^1.1.1",
"@remix-run/node": "^1.1.1", "@remix-run/node": "^1.1.1",
"@testing-library/cypress": "^8.0.2",
"@types/node": "*", "@types/node": "*",
"@types/react": "^17.0.38", "@types/react": "^17.0.38",
"@types/react-dom": "^17.0.11", "@types/react-dom": "^17.0.11",
"@types/tailwindcss": "^3.0.2", "@types/tailwindcss": "^3.0.2",
"@types/wait-on": "^5.3.1",
"concurrently": "^7.0.0", "concurrently": "^7.0.0",
"rehype-prism-plus": "^1.2.2", "cypress": "^9.2.1",
"execa": "^6.0.0",
"rehype-prism-plus": "^1.3.1",
"typedoc": "^0.22.10", "typedoc": "^0.22.10",
"typescript": "^4.5.4" "typescript": "^4.5.4",
"wait-on": "^6.0.0"
}, },
"engines": { "engines": {
"node": ">=14" "node": ">=14"

View File

Before

Width:  |  Height:  |  Size: 17 KiB

After

Width:  |  Height:  |  Size: 17 KiB

View File

@@ -0,0 +1,3 @@
{
"type": "module"
}

View File

@@ -0,0 +1,10 @@
import cypress from "cypress"
import { execa } from "execa"
import waitOn from "wait-on"
await execa("pnpm", ["build"], { stdio: "inherit" })
const app = execa("pnpm", ["start"], { stdio: "inherit" })
await waitOn({ resources: ["http-get://localhost:3000"] })
await cypress.run()
console.log("cypress run done")
app.kill()

2792
pnpm-lock.yaml generated

File diff suppressed because it is too large Load Diff

18
todo.md
View File

@@ -40,21 +40,24 @@
- instances - instances
- [x] sending channel messages - [x] sending channel messages
- [x] cleaning up instances - [x] cleaning up instances
- [ ] sending command replies - [x] sending command replies
- [ ] embeds - [x] embeds
- [ ] buttons and links - [x] buttons and links
- [ ] select menus - [x] select menus
- [ ] adapters
- api reference - api reference
- [x] rendering and making it available - [x] rendering and making it available
# docs polish # docs polish
- [ ] remove client-side react hydration - [x] remove client-side react hydration
- [ ] use a small script for the popover menu toggle - [x] ~~use a small script for the popover menu toggle~~ went with alpine
- [ ] adding doc comments to source - [ ] improve accessibility on mobile menu
- [x] adding doc comments to source
- [ ] docs: use literate-ts to typecheck code blocks - [ ] docs: use literate-ts to typecheck code blocks
- [ ] each page should have a link at the bottom to the previous and next pages - [ ] each page should have a link at the bottom to the previous and next pages
- [ ] anchor links on markdown headings - [ ] anchor links on markdown headings
- [ ] custom UI for api reference
# internal # internal
@@ -72,6 +75,7 @@
- [ ] timestamp component - [ ] timestamp component
- [ ] `useMessage` - [ ] `useMessage`
- [ ] `useReactions` - [ ] `useReactions`
- [ ] `useInstance` - returns the associated instance, probably replaces `useMessage`
- [ ] max instance count per guild - [ ] max instance count per guild
- [ ] max instance count per channel - [ ] max instance count per channel
- [ ] uncontrolled select - [ ] uncontrolled select

View File

@@ -10,6 +10,7 @@
"**/dist/**", "**/dist/**",
"**/.cache/**", "**/.cache/**",
"**/api/_build/**", "**/api/_build/**",
"**/public/**" "**/public/**",
"**/cypress/**"
] ]
} }