fix(slug): remove trailing hypens if any after reduction

This reverts commit cb3e46ca61.

.github/workflows/slugify-value.yaml

@@ -1,76 +1,179 @@
-name: Slugify testing
-on: [push]
+name: Slugify Value
+on:
+  push:
+    branches:
+      - v1.x
+  pull_request:
 jobs:
-  slugify-on-os:
+  os-testing:
     strategy:
       fail-fast: false
       matrix:
         os: [macos-latest, ubuntu-latest, windows-latest]
     runs-on: ${{ matrix.os }}
-    concurrency:
-      group: slugify-on-os-${{ github.ref }}
-      cancel-in-progress: true
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
 
       # Test 1
       - name: Slugify key only
+        id: slugify-key-only
         uses: ./
         with:
-          key: KEY_TEST
+          key: KEY_ONLY
         env:
-          KEY_TEST: Key_Test.values
+          KEY_ONLY: "refs/head/$-Key_Only.test--value-%-+"
       - name: Validate // Slugify key only
         run: |
-          [[ "${{ env.KEY_TEST }}" == "Key_Test.values" ]]
-          [[ "${{ env.KEY_TEST_SLUG }}" == "key_test.values" ]]
-          [[ "${{ env.KEY_TEST_SLUG_CS }}" == "Key_Test.values" ]]
-          [[ "${{ env.KEY_TEST_SLUG_URL }}" == "key_test-values" ]]
-          [[ "${{ env.KEY_TEST_SLUG_URL_CS }}" == "Key_Test-values" ]]
+          [[ "${{ env.KEY_ONLY }}" == "refs/head/$-Key_Only.test--value-%-+" ]]
+          [[ "${{ env.KEY_ONLY_SLUG }}" == "key_only.test--value" ]]
+          [[ "${{ env.KEY_ONLY_SLUG_CS }}" == "Key_Only.test--value" ]]
+          [[ "${{ env.KEY_ONLY_SLUG_URL }}" == "key-only-test--value" ]]
+          [[ "${{ env.KEY_ONLY_SLUG_URL_CS }}" == "Key-Only-test--value" ]]
+          [[ "${{ env.KEY_ONLY }}" == "${{ steps.slugify-key-only.outputs.value }}" ]]
+          [[ "${{ env.KEY_ONLY_SLUG }}" == "${{ steps.slugify-key-only.outputs.slug }}" ]]
+          [[ "${{ env.KEY_ONLY_SLUG_CS }}" == "${{ steps.slugify-key-only.outputs.slug-cs }}" ]]
+          [[ "${{ env.KEY_ONLY_SLUG_URL }}" == "${{ steps.slugify-key-only.outputs.slug-url }}" ]]
+          [[ "${{ env.KEY_ONLY_SLUG_URL_CS }}" == "${{ steps.slugify-key-only.outputs.slug-url-cs }}" ]]
         shell: bash
 
       # Test 2
       - name: Slugify key only with prefix
         uses: ./
         with:
-          key: KEY_TEST
+          key: KEY_ONLY_WITH_PREFIX
           prefix: CI_
         env:
-          KEY_TEST: Key_Test.values
+          KEY_ONLY_WITH_PREFIX: ${{ env.KEY_ONLY }}
       - name: Validate // Slugify key only with prefix
         run: |
-          [[ "${{ env.CI_KEY_TEST }}" == "${{ env.KEY_TEST }}" ]]
-          [[ "${{ env.CI_KEY_TEST_SLUG }}" == "${{ env.KEY_TEST_SLUG }}" ]]
-          [[ "${{ env.CI_KEY_TEST_SLUG_CS }}" == "${{ env.KEY_TEST_SLUG_CS }}" ]]
-          [[ "${{ env.CI_KEY_TEST_SLUG_URL }}" == "${{ env.KEY_TEST_SLUG_URL }}" ]]
-          [[ "${{ env.CI_KEY_TEST_SLUG_URL_CS }}" == "${{ env.KEY_TEST_SLUG_URL_CS }}" ]]
+          [[ "${{ env.CI_KEY_ONLY_WITH_PREFIX }}" == "${{ env.KEY_ONLY }}" ]]
+          [[ "${{ env.CI_KEY_ONLY_WITH_PREFIX_SLUG }}" == "${{ env.KEY_ONLY_SLUG }}" ]]
+          [[ "${{ env.CI_KEY_ONLY_WITH_PREFIX_SLUG_CS }}" == "${{ env.KEY_ONLY_SLUG_CS }}" ]]
+          [[ "${{ env.CI_KEY_ONLY_WITH_PREFIX_SLUG_URL }}" == "${{ env.KEY_ONLY_SLUG_URL }}" ]]
+          [[ "${{ env.CI_KEY_ONLY_WITH_PREFIX_SLUG_URL_CS }}" == "${{ env.KEY_ONLY_SLUG_URL_CS }}" ]]
         shell: bash
 
       # Test 3
       - name: Slugify key/value
         uses: ./
         with:
-          key: KEY_VALUE_TEST
-          value: refs/pulls/feat/-----Some----Changes_to.be------
+          key: KEY_VALUE
+          value: "Never gonna give you up Never gonna let you down"
       - name: Validate // Slugify key/value
         run: |
-          [[ "${{ env.KEY_VALUE_TEST }}" == "refs/pulls/feat/-----Some----Changes_to.be------" ]]
-          [[ "${{ env.KEY_VALUE_TEST_SLUG }}" == "feat-some-changes_to.be" ]]
-          [[ "${{ env.KEY_VALUE_TEST_SLUG_CS }}" == "feat-Some-Changes_to.be" ]]
-          [[ "${{ env.KEY_VALUE_TEST_SLUG_URL }}" == "feat-some-changes_to-be" ]]
-          [[ "${{ env.KEY_VALUE_TEST_SLUG_URL_CS }}" == "feat-Some-Changes_to-be" ]]
+          [[ "${{ env.KEY_VALUE }}" == "Never gonna give you up Never gonna let you down" ]]
+          [[ "${{ env.KEY_VALUE_SLUG }}" == "never-gonna-give-you-up-never-gonna-let-you-down" ]]
+          [[ "${{ env.KEY_VALUE_SLUG_CS }}" == "Never-gonna-give-you-up-Never-gonna-let-you-down" ]]
+          [[ "${{ env.KEY_VALUE_SLUG_URL }}" == "never-gonna-give-you-up-never-gonna-let-you-down" ]]
+          [[ "${{ env.KEY_VALUE_SLUG_URL_CS }}" == "Never-gonna-give-you-up-Never-gonna-let-you-down" ]]
         shell: bash
 
-  slugify-release:
+      # Test 4
+      - name: Slugify with another max length
+        uses: ./
+        with:
+          key: ANOTHER_MAX_LENGTH
+          value: "Never gonna give you up Never gonna let you down"
+          slug-maxlength: 24
+      - name: Validate // Slugify with another max length
+        run: |
+          [[ "${{ env.ANOTHER_MAX_LENGTH }}" == "Never gonna give you up Never gonna let you down" ]]
+          [[ "${{ env.ANOTHER_MAX_LENGTH_SLUG }}" == "never-gonna-give-you-up" ]]
+          [[ "${{ env.ANOTHER_MAX_LENGTH_SLUG_CS }}" == "Never-gonna-give-you-up" ]]
+          [[ "${{ env.ANOTHER_MAX_LENGTH_SLUG_URL }}" == "never-gonna-give-you-up" ]]
+          [[ "${{ env.ANOTHER_MAX_LENGTH_SLUG_URL_CS }}" == "Never-gonna-give-you-up" ]]
+        shell: bash
+
+      # Test 5
+      - name: Slugify with no limit on max length
+        uses: ./
+        with:
+          key: NO_MAX_LENGTH
+          value: "Never gonna give you up Never gonna let you down Never gonna run around and desert you Never gonna make you cry Never gonna say goodbye Never gonna tell a lie and hurt you"
+          slug-maxlength: "nolimit"
+      - name: Validate // Slugify with no limit on max length
+        run: |
+          [[ "${{ env.NO_MAX_LENGTH }}" == "Never gonna give you up Never gonna let you down Never gonna run around and desert you Never gonna make you cry Never gonna say goodbye Never gonna tell a lie and hurt you" ]]
+          [[ "${{ env.NO_MAX_LENGTH_SLUG }}" == "never-gonna-give-you-up-never-gonna-let-you-down-never-gonna-run-around-and-desert-you-never-gonna-make-you-cry-never-gonna-say-goodbye-never-gonna-tell-a-lie-and-hurt-you" ]]
+          [[ "${{ env.NO_MAX_LENGTH_SLUG_CS }}" == "Never-gonna-give-you-up-Never-gonna-let-you-down-Never-gonna-run-around-and-desert-you-Never-gonna-make-you-cry-Never-gonna-say-goodbye-Never-gonna-tell-a-lie-and-hurt-you" ]]
+          [[ "${{ env.NO_MAX_LENGTH_SLUG_URL }}" == "never-gonna-give-you-up-never-gonna-let-you-down-never-gonna-run-around-and-desert-you-never-gonna-make-you-cry-never-gonna-say-goodbye-never-gonna-tell-a-lie-and-hurt-you" ]]
+          [[ "${{ env.NO_MAX_LENGTH_SLUG_URL_CS }}" == "Never-gonna-give-you-up-Never-gonna-let-you-down-Never-gonna-run-around-and-desert-you-Never-gonna-make-you-cry-Never-gonna-say-goodbye-Never-gonna-tell-a-lie-and-hurt-you" ]]
+        shell: bash
+
+      # Test 6
+      - name: Slugify with empty max length
+        id: slugify-with-empty-max-length
+        uses: ./
+        with:
+          key: EMPTY_MAXLENGTH
+          value: some_value
+          slug-maxlength: ""
+        continue-on-error: true
+      - name: Validate // Slugify with empty max length
+        run: |
+          [[ -z "${{ env.EMPTY_MAXLENGTH }}" ]]
+          [[ -z "${{ env.EMPTY_MAXLENGTH_SLUG }}" ]]
+          [[ -z "${{ env.EMPTY_MAXLENGTH_SLUG_CS }}" ]]
+          [[ -z "${{ env.EMPTY_MAXLENGTH_SLUG_URL }}" ]]
+          [[ -z "${{ env.EMPTY_MAXLENGTH_SLUG_URL_CS }}" ]]
+          [[ "${{ steps.slugify-with-empty-max-length.outcome }}" == "failure" ]]
+          [[ "${{ steps.slugify-with-empty-max-length.conclusion }}" == "success" ]]
+        shell: bash
+
+      # Test 7
+      - name: Slugify with wrong max length
+        id: slugify-with-wrong-max-length
+        uses: ./
+        with:
+          key: WRONG_MAXLENGTH
+          value: some_value
+          slug-maxlength: "wrong"
+        continue-on-error: true
+      - name: Validate // Slugify with wrong max length
+        run: |
+          [[ -z "${{ env.WRONG_MAXLENGTH }}" ]]
+          [[ -z "${{ env.WRONG_MAXLENGTH_SLUG }}" ]]
+          [[ -z "${{ env.WRONG_MAXLENGTH_SLUG_CS }}" ]]
+          [[ -z "${{ env.WRONG_MAXLENGTH_SLUG_URL }}" ]]
+          [[ -z "${{ env.WRONG_MAXLENGTH_SLUG_URL_CS }}" ]]
+          [[ "${{ steps.slugify-with-wrong-max-length.outcome }}" == "failure" ]]
+          [[ "${{ steps.slugify-with-wrong-max-length.conclusion }}" == "success" ]]
+        shell: bash
+
+      # Test 8
+      - name: Slugify key without env publication
+        id: slugify-key-without-env-publication
+        uses: ./
+        with:
+          key: KEY_WITHOUT_ENV_PUBLICATION
+          publish-env: false
+        env:
+          KEY_WITHOUT_ENV_PUBLICATION: "Never gonna give you up"
+      - name: Validate // Slugify key without env publication
+        run: |
+          [[ -z "${{ env.KEY_WITHOUT_ENV_PUBLICATION }}" ]]
+          [[ -z "${{ env.KEY_WITHOUT_ENV_PUBLICATION_SLUG }}" ]]
+          [[ -z "${{ env.KEY_WITHOUT_ENV_PUBLICATION_SLUG_CS }}" ]]
+          [[ -z "${{ env.KEY_WITHOUT_ENV_PUBLICATION_SLUG_URL }}" ]]
+          [[ -z "${{ env.KEY_WITHOUT_ENV_PUBLICATION_SLUG_URL_CS }}" ]]
+          [[ "${{ steps.slugify-key-without-env-publication.outputs.value }}" == "Never gonna give you up" ]]
+          [[ "${{ steps.slugify-key-without-env-publication.outputs.slug }}" == "never-gonna-give-you-up" ]]
+          [[ "${{ steps.slugify-key-without-env-publication.outputs.slug-cs }}" == "Never-gonna-give-you-up" ]]
+          [[ "${{ steps.slugify-key-without-env-publication.outputs.slug-url }}" == "never-gonna-give-you-up" ]]
+          [[ "${{ steps.slugify-key-without-env-publication.outputs.slug-url-cs }}" == "Never-gonna-give-you-up" ]]
+        shell: bash
+
+  release:
     runs-on: ubuntu-latest
     concurrency:
-      group: slugify-release-${{ github.ref }}
-    needs: slugify-on-os
+      group: release-${{ github.ref }}
+    needs:
+      - os-testing
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
 
       # Release
       - name: Release this GitHub Action
-        uses: rlespinasse/release-that@v1.x
+        uses: rlespinasse/release-that@v1

README.md

@@ -8,8 +8,9 @@ Produce some `slug`-ed environment variables based on the input one.
 
   - put the variable content in lower case
   - replace any character by `-` except `0-9`, `a-z`, `.`, and `_`
-  - remove leading and trailing `-` character
+  - remove leading `-` character
   - limit the string size to 63 characters
+  - remove trailing `-` character
 
 - `<env name>_SLUG_CS`
 
@@ -17,7 +18,7 @@ Produce some `slug`-ed environment variables based on the input one.
 
 - `<env name>_SLUG_URL` (or `<env name>_SLUG_URL_CS`)
 
-  - like `<env name>_SLUG` (or `<env name>_SLUG_CS`) with the `.` character also replaced by `-`
+  - like `<env name>_SLUG` (or `<env name>_SLUG_CS`) with the `.`, and `_` characters also replaced by `-`
 
 ## Usage
 
@@ -70,10 +71,61 @@ Produce some `slug`-ed environment variables based on the input one.
   - `CI_EXISTING_ENV_VAR_SLUG_URL`
   - `CI_EXISTING_ENV_VAR_SLUG_URL_CS`
 
+- Slugify a value with a different slug length
+
+  ```yaml
+  - uses: rlespinasse/slugify-value@v1.x
+    with:
+      key: EXISTING_ENV_VAR
+      slug-maxlength: 80
+  ```
+
+  Will produce SLUG variables with a 80-char length
+
+- Slugify a value without length limit
+
+  ```yaml
+  - uses: rlespinasse/slugify-value@v1.x
+    with:
+      key: EXISTING_ENV_VAR
+      slug-maxlength: "nolimit"
+  ```
+
+  Will produce SLUG variables without limiting the output length
+
+- Slugify a value without publishing the environment variables
+
+  ```yaml
+  - uses: rlespinasse/slugify-value@v1.x
+    with:
+      key: KEY_NAME
+      value: value_to_slugify
+  ```
+
+  Will **not** make available
+
+  - `KEY_NAME`
+  - `KEY_NAME_SLUG`
+  - `KEY_NAME_SLUG_CS`
+  - `KEY_NAME_SLUG_URL`
+  - `KEY_NAME_SLUG_URL_CS`
+
 ## Inputs
 
-### `prefix`
+| Input          | Description                                                                                           | Mandatory | Default |
+| -------------- | ----------------------------------------------------------------------------------------------------- | --------- | ------- |
+| key            | Environment variable that will hold the value and serve as prefix to slugified value                  | Yes       |         |
+| value          | The value to slugify. If not set the value will be taken from the `key` input as environment variable | No        |         |
+| prefix         | The value will be prepend to each generated variable                                                  | No        |         |
+| slug-maxlength | The value is a number or `nolimit` to reflect the length of the slug outputs                          | No        | 63      |
+| publish-env    | In addition of the action output, the slug values are publish as environment variables                | No        | true    |
 
-The value will be prepend to each generated variable.
+## Outputs
 
-This input is _Optional_.
+| Output      | Description                     |
+| ----------- | ------------------------------- |
+| value       | The value to be slugify         |
+| slug        | Value Slug                      |
+| slug-cs     | Value Slug (Case-sensitive)     |
+| slug-url    | Value Slug URL                  |
+| slug-url-cs | Value Slug URL (Case-sensitive) |

action.yml

@@ -1,6 +1,9 @@
 name: "Slugify Value"
 description: "Github Action to slugify a value"
 author: "Romain Lespinasse"
+branding:
+  icon: "crop"
+  color: "gray-dark"
 inputs:
   key:
     description: "Environment variable that will hold the value and serve as prefix to slugified value"
@@ -12,15 +15,39 @@ inputs:
     description: "Value to prepend to each generated variable"
     default: ""
     required: false
-branding:
-  icon: "crop"
-  color: "gray-dark"
+  slug-maxlength:
+    description: "Max length of the slugified values"
+    default: "63"
+    required: true
+  publish-env:
+    description: "Publish slugs as environment variables"
+    default: "true"
+    required: true
+outputs:
+  value:
+    description: "The value to be slugify"
+    value: ${{ steps.slugify.outputs.value }}
+  slug:
+    description: "Value Slug"
+    value: ${{ steps.slugify.outputs.slug }}
+  slug-cs:
+    description: "Value Slug (Case-sensitive)"
+    value: ${{ steps.slugify.outputs.slug-cs }}
+  slug-url:
+    description: "Value Slug URL"
+    value: ${{ steps.slugify.outputs.slug-url }}
+  slug-url-cs:
+    description: "Value Slug URL (Case-sensitive)"
+    value: ${{ steps.slugify.outputs.slug-url-cs }}
 runs:
   using: "composite"
   steps:
-    - run: $GITHUB_ACTION_PATH/slugify.sh
+    - id: slugify
+      run: $GITHUB_ACTION_PATH/slugify.sh
       shell: bash
       env:
         INPUT_KEY: ${{ inputs.key }}
         INPUT_VALUE: ${{ inputs.value }}
         INPUT_PREFIX: ${{ inputs.prefix }}
+        INPUT_SLUG_MAXLENGTH: ${{ inputs.slug-maxlength }}
+        INPUT_PUBLISH_ENV: ${{ inputs.publish-env }}

slugify.sh

@@ -14,22 +14,61 @@ else
   PREFIX=${INPUT_PREFIX^^}
 fi
 
+MAX_LENGTH=""
+if [ -z "${INPUT_SLUG_MAXLENGTH}" ]; then
+  echo "::error ::slug-maxlength cannot be empty"
+  exit 1
+elif [ "${INPUT_SLUG_MAXLENGTH}" -eq "${INPUT_SLUG_MAXLENGTH}" ] 2>/dev/null; then
+  MAX_LENGTH="${INPUT_SLUG_MAXLENGTH}"
+elif [ "${INPUT_SLUG_MAXLENGTH}" == "nolimit" ]; then
+  MAX_LENGTH="${INPUT_SLUG_MAXLENGTH}"
+else
+  echo "::error ::slug-maxlength must be a number or equals to 'nolimit'"
+  exit 1
+fi
+
 slug() {
-  echo "$1" |
-    sed -E 's#refs/[^\/]*/##;s/[^a-zA-Z0-9._]+/-/g;s/-+/-/g;s/^-*//;s/-*$//' |
-    cut -c1-63
+  # 1st : Remove refs prefix
+  # 2d : Replace unwanted characters
+  # 3d : Remove leading hypens
+  output=$(sed -E 's#refs/[^\/]*/##;s/[^a-zA-Z0-9._-]+/-/g;s/^-*//' <<<"$1")
+  reduce "$output"
 }
 
 slug_url() {
-  echo "$1" |
-    sed -E 's#refs/[^\/]*/##;s/[^a-zA-Z0-9_]+/-/g;s/-+/-/g;s/^-*//;s/-*$//' |
-    cut -c1-63
+  # 1st : Remove refs prefix
+  # 2d : Replace unwanted characters
+  # 3d : Remove leading hypens
+  output=$(sed -E 's#refs/[^\/]*/##;s/[^a-zA-Z0-9-]+/-/g;s/^-*//' <<<"$1")
+  reduce "$output"
 }
 
-{
-  echo "${PREFIX}${KEY}=${CS_VALUE}"
-  echo "${PREFIX}${KEY}_SLUG=$(slug "$VALUE")"
-  echo "${PREFIX}${KEY}_SLUG_CS=$(slug "$CS_VALUE")"
-  echo "${PREFIX}${KEY}_SLUG_URL=$(slug_url "$VALUE")"
-  echo "${PREFIX}${KEY}_SLUG_URL_CS=$(slug_url "$CS_VALUE")"
-} >>"$GITHUB_ENV"
+reduce() {
+  reduced_value="$1"
+  if [ "${MAX_LENGTH}" != "nolimit" ]; then
+    reduced_value=$(cut -c1-"${MAX_LENGTH}" <<<"$reduced_value")
+  fi
+  # 1st : Remove trailing hypens
+  sed -E 's/-*$//' <<<"$reduced_value"
+}
+
+SLUG_VALUE=$(slug "$VALUE")
+SLUG_CS_VALUE=$(slug "$CS_VALUE")
+SLUG_URL_VALUE=$(slug_url "$VALUE")
+SLUG_URL_CS_VALUE=$(slug_url "$CS_VALUE")
+
+echo "::set-output name=value::${CS_VALUE}"
+echo "::set-output name=slug::${SLUG_VALUE}"
+echo "::set-output name=slug-cs::${SLUG_CS_VALUE}"
+echo "::set-output name=slug-url::${SLUG_URL_VALUE}"
+echo "::set-output name=slug-url-cs::${SLUG_URL_CS_VALUE}"
+
+if [ "${INPUT_PUBLISH_ENV}" == "true" ]; then
+  {
+    echo "${PREFIX}${KEY}=${CS_VALUE}"
+    echo "${PREFIX}${KEY}_SLUG=${SLUG_VALUE}"
+    echo "${PREFIX}${KEY}_SLUG_CS=${SLUG_CS_VALUE}"
+    echo "${PREFIX}${KEY}_SLUG_URL=${SLUG_URL_VALUE}"
+    echo "${PREFIX}${KEY}_SLUG_URL_CS=${SLUG_URL_CS_VALUE}"
+  } >>"$GITHUB_ENV"
+fi