fix sern build option

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

CHANGELOG.md

@@ -2,6 +2,20 @@
 
 All notable changes to this project will be documented in this file.
 
+## [1.2.1](https://github.com/sern-handler/cli/compare/v1.2.0...v1.2.1) (2024-02-08)
+
+
+### Bug Fixes
+
+* better error messages for publish ([d1832e4](https://github.com/sern-handler/cli/commit/d1832e44ce1b10aeb5b9dc902b7d35ab51c41ff3))
+
+## [1.2.0](https://github.com/sern-handler/cli/compare/v1.1.0...v1.2.0) (2024-01-29)
+
+
+### Features
+
+* make application id optional, thanks [@trueharuu](https://github.com/trueharuu) ([#130](https://github.com/sern-handler/cli/issues/130)) ([2d6e65a](https://github.com/sern-handler/cli/commit/2d6e65a1e6073f605aab192b8cea33037a04af2c))
+
 ## [1.1.0](https://github.com/sern-handler/cli/compare/v1.0.3...v1.1.0) (2024-01-28)
 
 

README.md

@@ -3,11 +3,11 @@
 </div>
 
 # Features
-
+- Manage discord application commands from the command line.
-😁 **User Friendly** <br>
+- Install plugins from the community.
-💦 **Simple** <br>
+- Really fast startup times (I think).
-🌱 **Efficient** <br>
+- Deploy with premade docker configurations.
-💪 **Powerful** <br>
+- Inhouse build tool based on esbuild built for sern applications, nearly **zero** config.
 
 ## Installation
 

package-lock.json

@@ -1,12 +1,12 @@
 {
     "name": "@sern/cli",
-    "version": "1.1.0",
+    "version": "1.2.1",
     "lockfileVersion": 3,
     "requires": true,
     "packages": {
         "": {
             "name": "@sern/cli",
-            "version": "1.1.0",
+            "version": "1.2.1",
             "license": "MIT",
             "dependencies": {
                 "@esbuild-kit/cjs-loader": "^2.4.2",

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@sern/cli",
-    "version": "1.1.0",
+    "version": "1.2.1",
     "description": "Official CLI for @sern/handler",
     "exports": "./dist/index.js",
     "bin": {

src/commands/command-clear.ts

@@ -23,9 +23,7 @@ const getConfirmation = (args: Record<string,any> ) => {
 export async function commandClear(args: Record<string,any>) {
     dotenv.configDotenv({ path: args.env || resolve('.env') })
     const token = process.env.token || process.env.DISCORD_TOKEN;
-    const appid = process.env.applicationId || process.env.APPLICATION_ID;
     assert(token, 'Could not find a token for this bot in .env or commandline. Do you have DISCORD_TOKEN in env?');
-    assert(appid, 'Could not find an application id for this bot in .env or commandline. Do you have APPLICATION_ID in env?');
     
     const confirmation = await getConfirmation(args);
    
@@ -34,7 +32,7 @@ export async function commandClear(args: Record<string,any>) {
             text: `Deleting ALL application commands...`,
             spinner: 'aesthetic',
         }).start();
-        const rest = Rest.create(appid, token);
+        const rest = await Rest.create(token);
         let guildCommands: Record<GuildId, CommandData[]> 
         try {
             guildCommands = JSON.parse(readFileSync('.sern/command-data-remote.json', 'utf-8'))

src/commands/publish.ts

@@ -15,7 +15,7 @@ export async function publish(commandDir: string | undefined, args: Partial<Publ
     args.import ??= [];
 
     args.token && console.info('Token passed through command line');
-    args.applicationId && console.info('applicationId passed through command line');
+    args.applicationId && console.info(magentaBright('WARNING')+ ' This option is deprecated. Do not pass applicationId through command line');
     commandDir && console.info('Publishing with override path: ', commandDir);
 
     const dotenvLocation = new URL('../node_modules/dotenv/config.js', rootPath),

src/create-publish.mts

@@ -10,7 +10,8 @@ import { once } from 'node:events';
 import * as Rest from './rest';
 import type { sernConfig } from './utilities/getConfig';
 import type { PublishableData, PublishableModule, Typeable } from './create-publish.d.ts';
-import { cyanBright, greenBright, redBright } from 'colorette';
+import { cyanBright, greenBright, magentaBright, redBright } from 'colorette';
+import { inspect } from 'node:util'
 import ora from 'ora';
 
 async function deriveFileInfo(dir: string, file: string) {
@@ -143,6 +144,7 @@ const serialize = (permissions: unknown) => {
 
 const makePublishData = ({ commandModule, config }: Record<string, Record<string, unknown>>) => {
     const applicationType = intoApplicationType(commandModule.type as number);
+    console.log(config)
     return {
         data: {
             name: commandModule.name as string,
@@ -152,6 +154,19 @@ const makePublishData = ({ commandModule, config }: Record<string, Record<string
             options: optionsTransformer((commandModule?.options ?? []) as Typeable[]),
             dm_permission: config?.dmPermission,
             default_member_permissions: serialize(config?.defaultMemberPermissions),
+            //@ts-ignore
+            integration_types: (config?.integrationTypes ?? ['Guild']).map(
+                (s: string) => {
+                    if(s === "Guild") {
+                        return 0
+                    } else if (s == "User") {
+                        return 1
+                    } else {
+                        throw Error("IntegrationType is not one of Guild or User");
+                    }
+                }),
+            //@ts-ignore
+            contexts: config?.contexts ? config.contexts : undefined
         },
         config,
     };
@@ -163,13 +178,13 @@ const publishableData = modules.map(makePublishData),
     appid = process.env.applicationId || process.env.APPLICATION_ID;
 
 assert(token, 'Could not find a token for this bot in .env or commandline. Do you have DISCORD_TOKEN in env?');
-assert(appid, 'Could not find an application id for this bot in .env or commandline. Do you have APPLICATION_ID in env?');
+appid && console.warn(`${magentaBright('WARNING')}: APPLICATION_ID is not necessary anymore`);
-
 // partition globally published and guilded commands
 const [globalCommands, guildedCommands] = publishableData.reduce(
     ([globals, guilded], module) => {
-        const isPublishableGlobally = !module.config || !Array.isArray(module.config.guildIds);
+        const isPublishableGlobally =  !Array.isArray(module.config?.guildIds);
         if (isPublishableGlobally) {
+            console.log(module)
             return [[module, ...globals], guilded];
         }
         return [globals, [module, ...guilded]];
@@ -181,36 +196,40 @@ const spin = ora(`Publishing ${cyanBright('Global')} commands`);
 
 globalCommands.length && spin.start();
 
-const rest = Rest.create(appid, token);
+const rest = await Rest.create(token);
 const res = await rest.updateGlobal(globalCommands);
 
 let globalCommandsResponse: unknown;
 
-
 if (res.ok) {
     globalCommands.length && spin.succeed(`All ${cyanBright('Global')} commands published`);
     globalCommandsResponse = await res.json();
+    console.log(globalCommandsResponse)
 } else {
     spin.fail(`Failed to publish global commands [Code: ${redBright(res.status)}]`);
-    switch(res.status) {
+    let err: Error
-        case 400 : 
-            throw Error("400: Ensure your commands have proper fields and data with nothing left out");
-        case 404 : 
-            throw Error("Forbidden 404. Is you application id and/or token correct?")
-        case 429: 
-            throw Error('Chill out homie, too many requests')
-    }
-    console.error('errors:',
-        await res
-            .json()
-            .then((res) => {
-                const errors = Object.values(res?.errors ?? {});
-                // @ts-ignore
-                return errors.map((err) => err?.name?._errors);
-            })
-            .catch(() => "No errors found (Unparsable json for a request with bad status code). Read the status code.")
-    );
     console.error("Status Text ", res.statusText);
+    switch(res.status) {
+        case 400 : {
+            const validation_errors = await res.json()
+            console.error('errors:', inspect(validation_errors, { depth: Infinity }));
+            console.error("Modules with validation errors:" 
+                          + inspect(Object.keys(validation_errors.errors).map(idx => globalCommands[idx as any])))
+            throw Error("400: Ensure your commands have proper fields and data with nothing left out");
+        }
+        case 404 : {
+            console.error('errors:', inspect(await res.json(), { depth: Infinity }));
+            throw Error("Forbidden 404. Is you application id and/or token correct?")
+        } 
+        case 429: {
+            console.error('errors:', inspect(await res.json(), { depth: Infinity }));
+            err = Error('Chill out homie, too many requests') 
+        } break;
+        default: {
+            console.error('errors:', inspect(await res.json(), { depth: Infinity }));
+            throw Error(res.status.toString() + " error")
+        }
+    }
 }
 
 function associateGuildIdsWithData(data: PublishableModule[]): Map<string, PublishableData[]> {
@@ -249,12 +268,21 @@ for (const [guildId, array] of guildCommandMap.entries()) {
     } else {
         spin.fail(`[${redBright(guildId)}] Failed to update commands for guild, Reason: ${result.message}`);
         switch(response.status) {
-            case 400 : 
+            case 400 : {
+                console.error(inspect(result, { depth: Infinity }))
+                console.error("Modules with validation errors:" 
+                + inspect(Object.keys(result.errors).map(idx => array[idx as any])))
+
                 throw Error("400: Ensure your commands have proper fields and data and nothing left out");
-            case 404 : 
+            }
+            case 404 : {
+                console.error(inspect(result, { depth: Infinity }))
                 throw Error("Forbidden 404. Is you application id and/or token correct?")
-            case 429: 
+            }
+            case 429: {
+                console.error(inspect(result, { depth: Infinity }))
                 throw Error('Chill out homie, too many requests')
+            }
         }
     }
 }

src/index.ts

@@ -39,8 +39,7 @@ program //
             .description('New way to manage your slash commands')
             .option('-W --suppress-warnings', 'suppress experimental warning')
             .option('-i, --import [scriptPath...]', 'Prerequire a script to load into publisher')
-            .option('-t, --token [token]')
+            .option('-e --env', 'path to .env file')
-            .option('--appId [applicationId]')
             .argument('[path]', 'path with respect to current working directory that will locate all published files')
             .action(async (...args) => importDynamic('publish.js').then((m) => m.publish(...args)))
     ).addCommand(
@@ -53,7 +52,6 @@ program //
             .option('-y, --yes', "Say yes to all prompts")
             .option('-e, --env [path]', "Supply a path to a .env")
             .action(async (...args) => importDynamic('command-clear.js').then((m) => m.commandClear(...args))));
-
 program
     .command('build')
     .description('Build your bot')

src/rest.ts

@@ -11,12 +11,22 @@ const publishablesIntoJson = (ps: PublishableModule[]) =>
         4
     );
 
-export const create = (appid: string, token: string) => {
+export const create = async (token: string) => {
-    const globalURL = new URL(`${appid}/commands`, baseURL);
     const headers = {
         Authorization: 'Bot ' + token,
         'Content-Type': 'application/json',
     };
+    let me;
+    let appid: string;
+    try {
+        me = await fetch(new URL('@me', baseURL), { headers }).then(res => res.json());
+        appid = me.id;
+    } catch(e) {
+        console.log("Something went wrong while trying to fetch your application:");
+        throw e;
+    }
+    const globalURL = new URL(`${appid}/commands`, baseURL);
+    
     return {
         updateGlobal: (commands: PublishableModule[]) =>
             fetch(globalURL, {

src/types/config.d.ts

@@ -13,7 +13,7 @@ export interface sernConfig {
 
 export interface TheoreticalEnv {
     DISCORD_TOKEN: string;
-    APPLICATION_ID: string;
+    APPLICATION_ID?: string;
     MODE: 'PROD' | 'DEV';
     [name: string]: string;
 }

src/utilities/getConfig.ts

@@ -11,7 +11,6 @@ export async function getConfig(): Promise<sernConfig> {
 
     return output;
 }
-
 export interface sernConfig {
     language: 'typescript' | 'javascript';
     defaultPrefix?: string;
@@ -20,5 +19,4 @@ export interface sernConfig {
         commands: string;
         events?: string;
     };
-    buildPath: string;
 }