convert the implementation of the action to TypeScript

.eslintignore

@@ -0,0 +1,4 @@
+dist/
+lib/
+node_modules/
+jest.config.js

.eslintrc.json

@@ -0,0 +1,55 @@
+{
+    "plugins": ["jest", "@typescript-eslint"],
+    "extends": ["plugin:github/recommended"],
+    "parser": "@typescript-eslint/parser",
+    "parserOptions": {
+      "ecmaVersion": 9,
+      "sourceType": "module",
+      "project": "./tsconfig.json"
+    },
+    "rules": {
+      "i18n-text/no-en": "off",
+      "eslint-comments/no-use": "off",
+      "import/no-namespace": "off",
+      "no-unused-vars": "off",
+      "@typescript-eslint/no-unused-vars": "error",
+      "@typescript-eslint/explicit-member-accessibility": ["error", {"accessibility": "no-public"}],
+      "@typescript-eslint/no-require-imports": "error",
+      "@typescript-eslint/array-type": "error",
+      "@typescript-eslint/await-thenable": "error",
+      "@typescript-eslint/ban-ts-comment": "error",
+      "camelcase": "off",
+      "@typescript-eslint/consistent-type-assertions": "error",
+      "@typescript-eslint/explicit-function-return-type": ["error", {"allowExpressions": true}],
+      "@typescript-eslint/func-call-spacing": ["error", "never"],
+      "@typescript-eslint/no-array-constructor": "error",
+      "@typescript-eslint/no-empty-interface": "error",
+      "@typescript-eslint/no-explicit-any": "error",
+      "@typescript-eslint/no-extraneous-class": "error",
+      "@typescript-eslint/no-for-in-array": "error",
+      "@typescript-eslint/no-inferrable-types": "error",
+      "@typescript-eslint/no-misused-new": "error",
+      "@typescript-eslint/no-namespace": "error",
+      "@typescript-eslint/no-non-null-assertion": "warn",
+      "@typescript-eslint/no-unnecessary-qualifier": "error",
+      "@typescript-eslint/no-unnecessary-type-assertion": "error",
+      "@typescript-eslint/no-useless-constructor": "error",
+      "@typescript-eslint/no-var-requires": "error",
+      "@typescript-eslint/prefer-for-of": "warn",
+      "@typescript-eslint/prefer-function-type": "warn",
+      "@typescript-eslint/prefer-includes": "error",
+      "@typescript-eslint/prefer-string-starts-ends-with": "error",
+      "@typescript-eslint/promise-function-async": "error",
+      "@typescript-eslint/require-array-sort-compare": "error",
+      "@typescript-eslint/restrict-plus-operands": "error",
+      "semi": "off",
+      "@typescript-eslint/semi": ["error", "never"],
+      "@typescript-eslint/type-annotation-spacing": "error",
+      "@typescript-eslint/unbound-method": "error"
+    },
+    "env": {
+      "node": true,
+      "es6": true,
+      "jest/globals": true
+    }
+  }

.gitattributes

@@ -0,0 +1 @@
+dist/** -diff linguist-generated=true 

.github/dependabot.yaml

@@ -2,7 +2,12 @@
 version: 2
 
 updates:
-  - package-ecosystem: "github-actions"
-    directory: "/"
+  - package-ecosystem: github-actions
+    directory: /
     schedule:
-      interval: "weekly"
+      interval: monthly
+
+  - package-ecosystem: npm
+    directory: /
+    schedule:
+      interval: monthly

.github/workflows/codeql.yml

@@ -0,0 +1,54 @@
+# For most projects, this workflow file will not need changing; you simply need
+# to commit it to your repository.
+#
+# You may wish to alter this file to override the set of languages analyzed,
+# or to provide custom queries or build logic.
+#
+# ******** NOTE ********
+# We have attempted to detect the languages in your repository. Please check
+# the `language` matrix defined below to confirm you have the correct set of
+# supported CodeQL languages.
+
+name: CodeQL
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    # The branches below must be a subset of the branches above
+    branches: [ main ]
+  schedule:
+    - cron: "0 0 * * 0" # Run every Sunday at 00:00.
+
+jobs:
+  analyze:
+    name: Analyze
+    runs-on: ubuntu-latest
+    permissions:
+      actions: read
+      contents: read
+      security-events: write
+
+    strategy:
+      fail-fast: false
+      matrix:
+        language: [ 'TypeScript' ]
+
+    steps:
+    - name: Checkout repository
+      uses: actions/checkout@v3
+
+    # Initializes the CodeQL tools for scanning.
+    - name: Initialize CodeQL
+      uses: github/codeql-action/init@v2
+      with:
+        languages: ${{ matrix.language }}
+        source-root: src
+
+    # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
+    # If this step fails, then you should remove it and run the build manually.
+    - name: Autobuild
+      uses: github/codeql-action/autobuild@v2
+
+    - name: Perform CodeQL Analysis
+      uses: github/codeql-action/analyze@v2

.github/workflows/dart.yml

@@ -6,10 +6,20 @@ on:
   pull_request:
     branches: [main]
   schedule:
-  - cron: "0 0 * * 0" # Run every Sunday at 00:00.
+    - cron: "0 0 * * 0" # Run every Sunday at 00:00.
 
 jobs:
 
+  # make sure build/ci work properly
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - run: |
+          npm install
+      - run: |
+          npm run all
+
   # Default test configurations.
   test:
     runs-on: ${{ matrix.os }}

.github/workflows/dist.yml

@@ -0,0 +1,55 @@
+# `dist/index.js` is a special file in Actions. When you reference an action
+# with `uses:` in a workflow, `index.js` is the code that will run.
+# 
+# For our project, we generate this file through a build process from other
+# source files. We need to make sure the checked-in `index.js` actually matches
+# what we expect it to be.
+
+name: Check dist/
+
+on:
+  push:
+    branches:
+      - main
+    paths-ignore:
+      - '**.md'
+  pull_request:
+    paths-ignore:
+      - '**.md'
+  workflow_dispatch:
+
+jobs:
+  check-dist:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Set Node.js 16.x
+        uses: actions/setup-node@v3.6.0
+        with:
+          node-version: 16.x
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Rebuild the dist/ directory
+        run: |
+          npm run build
+          npm run package
+
+      - name: Compare the expected and actual dist/ directories
+        run: |
+          if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
+            echo "Detected uncommitted changes after build.  See status below:"
+            git diff
+            exit 1
+          fi
+        id: diff
+
+      # If index.js was different than expected, upload the expected version as an artifact
+      - uses: actions/upload-artifact@v3
+        if: ${{ failure() && steps.diff.conclusion == 'failure' }}
+        with:
+          name: dist
+          path: dist/

.gitignore

@@ -1,3 +1,6 @@
-example/.packages
-example/pubspec.lock
-example/.dart_tool/
+# node modules
+node_modules/
+
+# typescript built files
+__tests__/runner/*
+lib/**/*

.prettierignore

@@ -0,0 +1,3 @@
+dist/
+lib/
+node_modules/

.prettierrc.json

@@ -0,0 +1,10 @@
+{
+  "printWidth": 80,
+  "tabWidth": 2,
+  "useTabs": false,
+  "semi": false,
+  "singleQuote": true,
+  "trailingComma": "none",
+  "bracketSpacing": false,
+  "arrowParens": "avoid"
+}

CHANGELOG.md

@@ -1,3 +1,10 @@
+## v1.5.0-dev
+
+* Re-wrote the implementation of the action into TypeScript.
+* Auto-detect the platform architecture (`x64`, `ia32`, `arm`, `arm64`).
+* Improve caching and download resilience of the sdk.
+* Added a new action output: `dart-version` - the installed version of the sdk.
+
 ## v1.4.0
 
 * Automatically create OIDC token for pub.dev.

CONTRIBUTING.md

@@ -1,8 +1,7 @@
-## How to Contribute
+# How to Contribute
 
-Prior to sending patches and contributions to this project, please file
-an issue in the issue tracker to ensure there is alignment with the overall
-goals of this project. Thanks!
+We'd love to accept your patches and contributions to this project. There are
+just a few small guidelines you need to follow.
 
 ## Contributor License Agreement
 
@@ -17,3 +16,56 @@ You generally only need to submit a CLA once, so if you've already submitted one
 (even if it was for a different project), you probably don't need to do it
 again.
 
+## Start with an issue
+
+Prior to sending patches and contributions to this project, please file an issue
+in the issue tracker to ensure there is alignment with the overall goals of this
+project. Thanks!
+
+## Code Reviews
+
+All submissions, including submissions by project members, require review. We
+use GitHub pull requests for this purpose. Consult
+[GitHub Help](https://help.github.com/articles/about-pull-requests/) for more
+information on using pull requests.
+
+## Coding style
+
+The Dart source code in this repo follows the:
+
+  * [Dart style guide](https://dart.dev/guides/language/effective-dart/style)
+
+You should familiarize yourself with those guidelines.
+
+## File headers
+
+All files in the Dart project must start with the following header; if you add a
+new file please also add this. The year should be a single number stating the
+year the file was created (don't use a range like "2011-2012"). Additionally, if
+you edit an existing file, you shouldn't update the year.
+
+    // Copyright (c) 2023, the Dart project authors.  Please see the AUTHORS file
+    // for details. All rights reserved. Use of this source code is governed by a
+    // BSD-style license that can be found in the LICENSE file.
+
+## Building and testing
+
+Install the dependencies:
+
+```bash
+$ npm install
+```
+
+Build the TypeScript code and package it for distribution:
+
+```bash
+$ npm run build && npm run package
+```
+
+## Community Guidelines
+
+This project follows
+[Google's Open Source Community Guidelines](https://opensource.google/conduct/).
+
+We pledge to maintain an open and welcoming environment. For details, see our
+[code of conduct](https://dart.dev/code-of-conduct).

README.md

@@ -36,8 +36,9 @@ The action takes the following inputs:
     * The available build flavors are `release` and `raw`.
     * The `release` flavor contains published builds.
     * The `raw` flavor contains unpublished builds; these can be used by
-      developers to test against SDK versions before a release. Note that the 
-      `main` release channel only supports `raw` build flavor.
+      developers to test against SDK versions before a signed release is
+      available. Note that the  `main` release channel only supports the `raw`
+      build flavor.
 
   * `architecture`: The CPU architecture to setup support for.
     * Valid options are `x64`, `ia32`, `arm`, and `arm64`.
@@ -46,7 +47,13 @@ The action takes the following inputs:
       [Dart system requirements](https://dart.dev/get-dart#system-requirements)
       for valid combinations.
 
-## Check static analysis, formatting, and run tests
+## Outputs
+
+The action produces the following output:
+
+  * `dart-version`: The version of the Dart SDK that was installed.
+
+## Checking static analysis, formatting, and running tests
 
 Various static checks:
 
@@ -171,4 +178,4 @@ Contributions are welcome! Please see [CONTRIBUTING.md.md](CONTRIBUTING.md.md).
 
 ## Version history
 
-Please see out [CHANGELOG.md](CHANGELOG.md) file.
+Please see our [CHANGELOG.md](CHANGELOG.md) file.

__tests__/main.test.ts

@@ -0,0 +1,10 @@
+import * as system from '../src/system'
+import { expect, test } from '@jest/globals'
+
+test('getPlatform', () => {
+  expect(['macos', 'linux', 'windows']).toContain(system.getPlatform())
+})
+
+test('getArch', () => {
+  expect(['x64', 'arm64']).toContain(system.getArch())
+})

action.yml

@@ -1,26 +1,26 @@
 name: "Setup Dart SDK"
-description: "Setup the Dart SDK, and add it to the PATH"
+description: "Download and setup the Dart SDK."
 branding:
   icon: check-circle
   color: blue
 inputs:
   sdk:
     description: >-
       The channel, or a specific version from a channel to install ('stable',
-      'beta', 'dev', '2.7.2', '2.12.0-1.4.beta'). Using one of the three
+      'beta', 'dev', '2.7.2', '2.12.0-1.4.beta'). Using one of the named
       channels instead of a version will give you the latest version published
       to that channel.
     required: false
     default: "stable"
   architecture:
     description: "The CPU architecture ('x64', 'ia32', 'arm', or 'arm64')."
     required: false
-    default: "x64"
   flavor:
-    description: "The build flavor ('raw' or 'release')."
+    description: "The build flavor ('release' or 'raw')."
     required: false
+outputs:
+  dart-version:
+    description: 'The installed Dart version.'
 runs:
-  using: "composite"
-  steps:
-    - run: $GITHUB_ACTION_PATH/setup.sh ${{ inputs.sdk }} ${{ runner.os }} ${{ inputs.architecture }} ${{ inputs.flavor }}
-      shell: bash
+  using: "node16"
+  main: "dist/index.js"