fix: gist_description will override the original description if not provided

Author: exuanbo

__tests__/index.test.ts

@@ -4,14 +4,15 @@ import dotenv from 'dotenv'
 import { getInput } from '../src/input'
 
 test('input', () => {
+  process.env.INPUT_TOKEN = 'token'
   process.env.INPUT_GIST_ID = 'gist_id'
   process.env.INPUT_GIST_DESCRIPTION = 'gist_description'
   process.env.INPUT_GIST_FILE_NAME = 'gist_file_name'
   process.env.INPUT_FILE_PATH = 'file_path'
 
   const input = getInput()
 
-  expect(input.token).toMatch('')
+  expect(input.token).toMatch('token')
   expect(input.gistId).toMatch('gist_id')
   expect(input.gistDescription).toMatch('gist_description')
   expect(input.gistFileName).toMatch('gist_file_name')
@@ -20,6 +21,7 @@ test('input', () => {
 
 test('run', () => {
   process.env.GITHUB_WORKSPACE = process.cwd()
+  delete process.env.INPUT_TOKEN
   dotenv.config()
   process.env.INPUT_GIST_ID = 'e885afa349a0e5d1cfb408e46d6a37bc'
   process.env.INPUT_GIST_DESCRIPTION = 'foo bar'

dist/index.js

@@ -1,19 +1,35 @@
-import { getInput as getActionInput } from '@actions/core'
+// https://github.com/actions/toolkit/blob/27f76dfe1afb2b7e5e679cd8e97192d34d8320e6/packages/core/src/core.ts#L128
+function getInputFromEnv(name: string, options: { required: true }): string
+function getInputFromEnv(
+  name: string,
+  options?: Record<string, never> | { required: false }
+): string | undefined
+function getInputFromEnv(
+  name: string,
+  options: { required?: boolean } = {}
+): string | undefined {
+  const value = process.env[`INPUT_${name.replace(/ /g, '_').toUpperCase()}`]
+  const { required = false } = options
+  if (required && value === undefined) {
+    throw new Error(`Input required and not supplied: ${name}`)
+  }
+  return value?.trim()
+}
 
 type Input = Readonly<{
   token: string
   gistId: string
-  gistDescription: string
-  gistFileName: string
+  gistDescription: string | undefined
+  gistFileName: string | undefined
   filePath: string
 }>
 
 export const getInput = (): Input => {
   return {
-    token: getActionInput('token'),
-    gistId: getActionInput('gist_id'),
-    gistDescription: getActionInput('gist_description'),
-    gistFileName: getActionInput('gist_file_name'),
-    filePath: getActionInput('file_path')
+    token: getInputFromEnv('token', { required: true }),
+    gistId: getInputFromEnv('gist_id', { required: true }),
+    gistDescription: getInputFromEnv('gist_description'),
+    gistFileName: getInputFromEnv('gist_file_name'),
+    filePath: getInputFromEnv('file_path', { required: true })
   }
 }

src/input.ts

@@ -9,15 +9,13 @@ export const run = async (): Promise<void> => {
 
   const workSpace = process.env.GITHUB_WORKSPACE!
   const filePath = path.join(workSpace, input.filePath)
-  const fileName =
-    input.gistFileName.length === 0
-      ? path.basename(filePath)
-      : input.gistFileName
+
+  const fileName = input.gistFileName ?? path.basename(filePath)
 
   startGroup('Dump inputs')
   info(`\
 [INFO] GistId: ${input.gistId}${
-    input.gistDescription.length === 0
+    input.gistDescription === undefined
       ? ''
       : `\n[INFO] GistDescription: ${input.gistDescription}`
   }