Issue with file path input

[tooomm]

What I want to archive

• Parse a file as input and dump the URLs into a md file
• Check the URLs from the dumped md file as input for lychee

See my config:

      # (1) Extract URLs
      - name: Extract URLs
        uses: lycheeverse/lychee-action@v2
        with:
          args: --dump --exclude www.w3.org -- input.xml
          output: lychee/url_list.md
          jobSummary: false

      # (2) Print output
      - name: Print List of URLs
        run: cat ./lychee/url_list.md

      # (3) Check dumped URLs from file
      - name: Check URLs
        uses: lycheeverse/lychee-action@v2
        with:
          args: --no-progress --require-https -- './lychee/url_list.md'
          jobSummary: true

✅ The (1) first step succeeds and the file is dumped.
✅ The (2) second steps succeeds and the dumped output file is found + printed.
❌ The (3) third step is failing and the dumped output file can not be found and used as input any more.

Error: Cannot parse inputs from arguments

Caused by:
    Invalid file path: ./lychee/url_list.md

Any idea?

I'm using the same file path in (2) and (3).
I also tried ./lychee/url_list.md without wrapping in ', which leads to the same error.

Originally posted by @tooomm in #214

[mre]

Did you try a debug run in GitHub Actions? Maybe it reveals more information.

[tooomm]

It's printing a lot of lines... but nothing catching my attention.
I did copy the output around the ARGs and the error below:

...
Run # Remove the downloaded archive and any unnecessary files after installation
##[debug]/usr/bin/bash --noprofile --norc -e -o pipefail /home/runner/work/_temp/def0094a-0b1e-49d5-9261-b8c0589a6e65.sh
##[debug]Finished: run
##[debug]Evaluating: inputs.TOKEN
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'TOKEN'
##[debug]=> '***'
##[debug]Result: '***'
##[debug]Evaluating: inputs.ARGS
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'ARGS'
##[debug]=> '--no-progress --require-https --cache --max-cache-age 8h -- ./lychee/url_list.md'
##[debug]Result: '--no-progress --require-https --cache --max-cache-age 8h -- ./lychee/url_list.md'
##[debug]Evaluating: inputs.DEBUG
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'DEBUG'
##[debug]=> 'false'
##[debug]Result: 'false'
##[debug]Evaluating: inputs.FAIL
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'FAIL'
##[debug]=> 'true'
##[debug]Result: 'true'
##[debug]Evaluating: inputs.FAILIFEMPTY
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'FAILIFEMPTY'
##[debug]=> 'true'
##[debug]Result: 'true'
##[debug]Evaluating: inputs.FORMAT
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'FORMAT'
##[debug]=> 'markdown'
##[debug]Result: 'markdown'
##[debug]Evaluating: inputs.JOBSUMMARY
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'JOBSUMMARY'
##[debug]=> 'true'
##[debug]Result: 'true'
##[debug]Evaluating: inputs.CHECKBOX
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'CHECKBOX'
##[debug]=> 'true'
##[debug]Result: 'true'
##[debug]Evaluating: inputs.OUTPUT
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'OUTPUT'
##[debug]=> 'lychee/out.md'
##[debug]Result: 'lychee/out.md'
##[debug]Evaluating condition for step: 'run'
##[debug]Evaluating: success()
##[debug]Evaluating success:
##[debug]=> true
##[debug]Result: true
##[debug]Starting: run
##[debug]Loading inputs
##[debug]Evaluating: format('{0}/entrypoint.sh', github.action_path)
##[debug]Evaluating format:
##[debug]..Evaluating String:
##[debug]..=> '{0}/entrypoint.sh'
##[debug]..Evaluating Index:
##[debug]....Evaluating github:
##[debug]....=> Object
##[debug]....Evaluating String:
##[debug]....=> 'action_path'
##[debug]..=> '/home/runner/work/_actions/lycheeverse/lychee-action/v2'
##[debug]=> '/home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh'
##[debug]Result: '/home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh'
##[debug]Evaluating: inputs.workingDirectory
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'workingDirectory'
##[debug]=> '.'
##[debug]Result: '.'
##[debug]Loading env
Run /home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh
##[debug]/usr/bin/bash --noprofile --norc -e -o pipefail /home/runner/work/_temp/8ebf6aff-f756-4b8e-9ce9-a1cec9eef789.sh
/home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh: line 48: [: : integer expression expected
Error: Cannot parse inputs from arguments

Caused by:
    Invalid file path: ./lychee/url_list.md

Error: Process completed with exit code 1.
##[debug]Set output exit_code = 1
##[debug]Finished: run
##[debug]Evaluating: steps.run-lychee.outputs.exit_code
##[debug]Evaluating Index:
##[debug]..Evaluating Index:
##[debug]....Evaluating Index:
##[debug]......Evaluating steps:
##[debug]......=> Object
##[debug]......Evaluating String:
##[debug]......=> 'run-lychee'
##[debug]....=> Object
##[debug]....Evaluating String:
##[debug]....=> 'outputs'
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'exit_code'
##[debug]=> '1'
##[debug]Result: '1'
##[debug]Finishing: Check token art URLs

[mre]

Ah, okay, that's helpful, thanks.

Here's a few things you could try:

Try using an absolute path:

It shouldn't matter, but maybe you can give this a shot:

- name: Check URLs
   uses: lycheeverse/lychee-action@v2
   with:
      args: --no-progress --require-https -- ${{ github.workspace }}/lychee/url_list.md
      jobSummary: true

Remove double-dash

This line in our entrypoint.sh script that could be causing issues:

eval lychee ${CHECKBOX} ${FORMAT} --output ${LYCHEE_TMP} ${ARGS}

The way arguments are evaluated here might be problematic. When using the double dash (--) as input separator, it might need special handling.

Can you try without the -- in your args?

Use the workingDirectory parameter:

If nothing else helps, you can also try the new workingDirectory param now.
It probably just causes more problems in your case, but it's worth a shot, because I'm kinda running out of ideas here. 😅

- name: Check URLs
   uses: lycheeverse/lychee-action@v2
   with:
      args: --no-progress --require-https -- url_list.md
      workingDirectory: ./lychee
      jobSummary: true

[tooomm]

No luck...

Try using an absolute path:

• name: Check URLs
  uses: lycheeverse/lychee-action@v2
  with:
  args: --no-progress --require-https -- ${{ github.workspace }}/lychee/url_list.md
  jobSummary: true

Run /home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh
/home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh: line 48: [: : integer expression expected
Error: Network error

Caused by:
0: error sending request for url (http://home/runner/work/[REPO]/[REPO]/lychee/url_list.md)
1: client error (Connect)
2: dns error: failed to lookup address information: Temporary failure in name resolution
3: failed to lookup address information: Temporary failure in name resolution

Remove double-dash

Can you try without the -- in your args?

Run /home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh
/home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh: line 48: [: : integer expression expected
Error: Cannot parse inputs from arguments

Caused by:
Invalid file path: ./lychee/url_list.md

Use the workingDirectory parameter:

• name: Check URLs
  uses: lycheeverse/lychee-action@v2
  with:
  args: --no-progress --require-https -- url_list.md
  workingDirectory: ./lychee
  jobSummary: true

Run /home/runner/work/_actions/lycheeverse/lychee-action/v2/entrypoint.sh
Error: An error occurred trying to start process '/usr/bin/bash' with working directory '/home/runner/work/[REPO]/[REPO]/./lychee'. No such file or directory

I'm also wondering why the repo name comes up twice in the path. 🤔
Would have expected /[ORG]/[REPO].

When trying with /lychee:
Error: An error occurred trying to start process '/usr/bin/bash' with working directory '/lychee'. No such file or directory

When trying with lychee:
Error: An error occurred trying to start process '/usr/bin/bash' with working directory '/home/runner/work/[REPO]/[REPO]/lychee'. No such file or directory

[mre]

Yeah, the last one is indeed pretty interesting. Sounds more related to the working directory arg, though.

Something is really off. To narrow down the problem, I would try to create a new repository with a very clean setup, which just contains a single workflow. It's a bit awkward, but I can't think of another way to narrow down the problem.

[tooomm]

I created a clean test repo and copied this repos README over: https://github.com/tooomm/lychee-scan-dump-test

Same issue there when checking with my starting config...
Feel free to have a look at the workflow file or create PR's/commits. Added you.

[mre]

Thanks! I think here's the issue: https://github.com/tooomm/lychee-scan-dump-test/pull/2#issuecomment-2800135048

[tooomm]

(copied your comment over)

lychee-action/action.yml

Line 106 in 6f793a7

rm -f lychee!(*-bin|*-lib|*.toml)

We clean up all folders starting with lychee in the lychee-action. Changing the dir fixed it. We should make that removal less restrictive and also document the behavior.

Thanks a lot for your investigation, @mre! ❤
Made my logic work with this slight changes and not using "lychee" folder to store the dumped output.

I suggest that the default output value is changed in that case (or the deletion logic).

The default value for output is lychee/out.md currently:
https://github.com/lycheeverse/lychee-action/blob/master/action.yml#L34

Also, the example in the README refers to lychee/results.md

[mre]

Yeah, I agree. Maybe we should use a random output file?
GitHub provides ${RUNNER_TEMP} for that. (See the reference).