Extension is not compatible with the last version of Quarto (1.8.34)

[jchiquet]

Running your example on the qmd files that you provide leads to the following with Quarto 1.8.24 (it perfectly works with Quarto 1.7.34, though).

simple|main ⇒ quarto render simple.qmd 
pandoc 
  to: html
  output-file: simple.html
  standalone: true
  section-divs: true
  html-math-method: mathjax
  wrap: none
  default-image-extension: png
  variables: {}
  
metadata
  document-css: false
  link-citations: true
  date-format: long
  lang: en
  title: Quarto Pseudocode Extension Example
  pseudocode:
    caption-prefix: Algorithm
    reference-prefix: Algorithm
    caption-number: true
  
FATAL (/opt/quarto/share/filters/main.lua:13615) An error occurred:
This is an internal error. Please file a bug report at https://github.com/quarto-dev/quarto-cli/
Error running filter /opt/quarto/share/filters/main.lua:
...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:34: FATAL QUARTO ERROR
stack traceback:
	/opt/quarto/share/filters/main.lua:2619: in function 'fatal'
	/opt/quarto/share/filters/main.lua:2199: in function 'fail'
	/opt/quarto/share/filters/main.lua:2204: in function 'internal_error'
	/opt/quarto/share/filters/main.lua:13615: in function </opt/quarto/share/filters/main.lua:13611>
	(...tail calls...)
	...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:34: in upvalue 'run_filter_function'
	...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:223: in function <...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:202>
	(...tail calls...)
	[C]: in function 'tmap'
	...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:125: in upvalue 'recurse'
	...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:222: in local 'jogger'
	...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:171: in upvalue 'recurse'
	...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:222: in function <...uarto/share/pandoc/datadir/../../filters/modules/jog.lua:202>
	(...tail calls...)
	/opt/quarto/share/filters/main.lua:572: in function </opt/quarto/share/filters/main.lua:550>
	(...tail calls...)
	/opt/quarto/share/filters/main.lua:1314: in local 'callback'
	/opt/quarto/share/filters/main.lua:1332: in upvalue 'run_emulated_filter_chain'
	/opt/quarto/share/filters/main.lua:1368: in function </opt/quarto/share/filters/main.lua:1365>
WARN: Should not have arrived here:
ERROR: [non-error-thrown] undefined

Stack trace:
    at _Command.handleError (file:///opt/quarto/bin/quarto.js:10032:117)
    at _Command.parseCommand (file:///opt/quarto/bin/quarto.js:9719:12)
    at eventLoopTick (ext:core/01_core.js:178:7)
    at async quarto (file:///opt/quarto/bin/quarto.js:175522:5)
    at async file:///opt/quarto/bin/quarto.js:175550:5
    at async file:///opt/quarto/bin/quarto.js:175405:14
    at async mainRunner (file:///opt/quarto/bin/quarto.js:175407:5)
    at async file:///opt/quarto/bin/quarto.js:175543:3

[mcanouil]

I believe the filter needs to run before "everything else" because of the way the special/custom cross-reference mechanism of this filter.

filters:
  - at: pre-ast
    path: pseudocode

The "built-in" (to this extension) cross-reference using alg- prefix is the thing causing issue because it conflicts with Quarto's internal cross-reference system.
Support for alg- should probably be with custom cross-reference float which would be handled fully by Quarto CLI itself and avoid other breakage like this in the future I believe.

[leovan]

alg is a reserved prefix for cross reference with quarto 1.8. I think making algo- be the build-in custom cross reference prefix will fix this.

[cderv]

Yes it was added in

• crossref - add support for algorithm (alg-) theorem quarto-dev/quarto-cli#12615

Sorry about this—we don't have a way to test every Quarto Extension out there yet. Maybe we could find a way based on an existing listing 🤷‍♂️.
However, I would suggest having a basic workflow file with a GitHub action in your repo to run the latest pre-release version as soon as something is breaking your product and report to us.

Also, I note that the current error thrown is not correct, in a way not expected, and horribly useless. So I'll look into this for better error.

[leovan]

Try the new release, and modify the code by:

1. Change the prefix from alg- to algo- when using build-in cross reference to make it compatible with Quarto 1.8.
2. Change the prefix from algo- to alg- when using Quarto custom cross reference to make it compatible with Quarto 1.8.