Add dictsort filter to @huggingface/jinja

[Copilot]

• Understand the dictsort filter specification from Jinja docs
• Implement the dictsort filter in runtime.ts for ObjectValue
• Add tests for dictsort filter with various parameters
• Test the implementation with edge cases
• Run lint and build to ensure code quality
• Refactor to eliminate code duplication by extracting helper method
• Improve null/undefined handling in sort comparisons
• Add documentation for type handling assumptions
• Move dictsort logic to ObjectValue builtins as FunctionValue per @xenova's feedback
• Extract index calculation to avoid duplication

Original prompt

On jinja, Add support for the dictsort. jinja filter to @huggingface/jinja package. Here are the docs for the feature: https://jinja.palletsprojects.com/en/stable/templates/#jinja-filters.dictsort

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[xenova]

@copilot This is a good start, but I'd prefer for the applyDictSort logic to be moved to the builtins section of the ObjectValue class, turning dictsort into a FunctionValue which can be called as a filter.

[Copilot]

@copilot This is a good start, but I'd prefer for the applyDictSort logic to be moved to the builtins section of the ObjectValue class, turning dictsort into a FunctionValue which can be called ...

Done! I've moved the dictsort logic to the builtins section of the ObjectValue class as a FunctionValue. The implementation now:

1. Adds dictsort as a builtin in ObjectValue (lines 293-397)
2. Checks for ObjectValue builtins in both identifier and call expression filter paths
3. Removes the applyDictSort helper method

All tests pass and the implementation is cleaner. Commit: 85481b0

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• huggingface.co

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[xenova]

Looks great 👍 minor nit.

[xenova]

@copilot we can reuse by.value === "key" ? 0 : 1

[Copilot]

Good catch! Extracted the index calculation to a variable. Commit: 9f169f1

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• huggingface.co
  • Triggering command: node /home/REDACTED/work/huggingface.js/huggingface.js/node_modules/.bin/../vitest/vitest.mjs run (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)