Enhances certificate manifest rendering and ORCID validation

Improves certificate generation by adding support for more image and data formats in the manifest, including GIF, TIF, TSV, and JSON.

Adds automatic conversion of GIF and TIF/TIFF images to PNG for better compatibility, along with more robust error handling to prevent failures when encountering corrupted or unsupported files.

Updates ORCID validation to handle authentication failures gracefully and allows skipping validation in non-interactive environments.

Requires `magick` package dependency for image conversion.

Author: nuest

CLAUDE.md

@@ -17,14 +17,26 @@ remotes::install_github("codecheckers/codecheck")
 ```
 
 Run tests using `tinytest`:
-```r
-# Interactive testing
-tinytest::test_all("/path/to/package")
 
-# Or build, install, and test in a fresh environment
+**⚠️ CRITICAL: ALWAYS use `build_install_test()` for testing this package!**
+
+```r
+# CORRECT: Build, install, and test in a fresh environment
 tinytest::build_install_test(".")
 ```
 
+**DO NOT use `test_all()` from the source directory** - it will cause test failures because the package functions won't be properly loaded. The error "could not find function X" indicates you're using the wrong test method.
+
+```r
+# WRONG - DO NOT USE: Will cause "could not find function" errors
+tinytest::test_all(".")  # ❌ DON'T DO THIS
+
+# For interactive testing of specific functions during development:
+# 1. First install the package: devtools::install()
+# 2. Then load it: library(codecheck)
+# 3. Then test interactively or source specific test files
+```
+
 The package uses GitHub Actions for CI/CD. Check `.github/workflows/R-CMD-check.yaml` for the automated testing setup.
 
 ### Changelog Management
@@ -66,8 +78,8 @@ When bumping the version in `DESCRIPTION`, add a new section to `NEWS.md` with t
 **External validation**: Ensure consistency with published paper metadata and ORCID records:
 
 - `validate_codecheck_yml_crossref()` - Validates paper metadata against CrossRef API; compares title and author information with CrossRef data
-- `validate_codecheck_yml_orcid()` - Validates author and codechecker names against ORCID API; queries ORCID records using rorcid package; compares names in ORCID records with local metadata
-- `validate_contents_references()` - Comprehensive validation wrapper; runs both CrossRef and ORCID validations; provides unified summary; supports strict mode for certificate rendering
+- `validate_codecheck_yml_orcid()` - Validates author and codechecker names against ORCID API; queries ORCID records using rorcid package; compares names in ORCID records with local metadata; requires ORCID authentication by default (set `skip_on_auth_error = TRUE` to skip validation when authentication is unavailable)
+- `validate_contents_references()` - Comprehensive validation wrapper; runs both CrossRef and ORCID validations; provides unified summary; supports strict mode for certificate rendering; requires ORCID authentication by default (users can opt-in to skipping via `skip_on_auth_error = TRUE`)
 
 **Zenodo integration**: Functions for uploading certificates to Zenodo:
 

DESCRIPTION

@@ -44,7 +44,8 @@ Imports:
     whisker,
     pdftools,
     readxl,
-    rsvg
+    rsvg,
+    magick
 Encoding: UTF-8
 LazyData: true
 RoxygenNote: 7.3.3

NAMESPACE

@@ -45,6 +45,8 @@ importFrom(httr,write_disk)
 importFrom(jsonlite,fromJSON)
 importFrom(jsonlite,prettify)
 importFrom(knitr,kable)
+importFrom(magick,image_read)
+importFrom(magick,image_write)
 importFrom(osfr,osf_download)
 importFrom(osfr,osf_ls_files)
 importFrom(osfr,osf_retrieve_node)

NEWS.md

@@ -2,21 +2,26 @@
 
 ## ORCID Validation Improvements
 
-* **Graceful authentication handling**: ORCID validation functions now handle authentication failures gracefully instead of requiring interactive login
-* **New `skip_on_auth_error` parameter**: Added to `validate_codecheck_yml_orcid()` and `validate_contents_references()` to control behavior when ORCID authentication is unavailable (defaults to `TRUE` for non-interactive environments)
-* **Enhanced error messages**: Clear guidance provided when ORCID authentication is needed, with instructions for setting `ORCID_TOKEN` environment variable
-* **Test and CI/CD compatibility**: Certificate template and all validation functions now work seamlessly in test and CI/CD environments without requiring ORCID authentication
-* **Backward compatibility**: Existing code continues to work without modification; validation automatically skips when authentication is unavailable
+* **Graceful authentication handling**: ORCID validation functions now handle authentication failures gracefully with clear error messages instead of requiring interactive login
+* **New `skip_on_auth_error` parameter**: Added to `validate_codecheck_yml_orcid()` and `validate_contents_references()` to control behavior when ORCID authentication is unavailable. By default (`FALSE`), functions require authentication. Set to `TRUE` to skip validation when authentication is not available (useful for CI/CD pipelines and test environments).
+* **Enhanced error messages**: Clear guidance provided when ORCID authentication is needed, with instructions for setting `ORCID_TOKEN` environment variable or running `rorcid::orcid_auth()`
+* **Opt-in skipping**: Certificate authors can choose to skip ORCID validation by setting `skip_on_auth_error = TRUE` in the certificate template
 * **Better feedback**: Functions now return a `skipped` field indicating whether validation was skipped due to authentication issues
 
 ## Manifest Rendering Enhancements
 
 * **Expanded format support**: Certificates can now render additional file formats in the manifest section:
-  - Image formats: TIF, TIFF, EPS, and SVG (with automatic PDF conversion)
+  - Image formats: TIF, TIFF, GIF, EPS, and SVG (with automatic conversion)
   - Data formats: JSON (with pretty-printing and configurable line limits) and TSV (tab-separated values)
   - Multi-page PDFs are now fully supported with automatic page detection
+* **GIF format support**: GIF images are now automatically converted to PNG during certificate rendering. pdflatex does not natively support GIF format, so conversion is required.
+* **TIF/TIFF format support**: TIF and TIFF images are now automatically converted to PNG during certificate rendering.
+* **New dependency**: Added `magick` package as a required dependency for image format conversion (TIF/TIFF/GIF to PNG). Previously this was optional, but is now mandatory for proper image format support.
+* **Graceful error handling**: Missing, corrupted, or unsupported files no longer fail the entire certificate rendering. Instead, a formatted error box is displayed in the PDF for each problematic file, allowing codecheckers to identify and fix issues without blocking certificate generation.
+* **File existence checks**: All manifest rendering functions now check for file existence before processing and display helpful error messages.
+* **Improved error messages**: Error messages are now displayed as formatted LaTeX boxes in the rendered PDF with specific information about what went wrong (e.g., "File not found", "Failed to convert GIF image", "Unsupported file format (.xyz)").
 * **Improved maintainability**: Manifest rendering code refactored into modular, testable components
-* **Comprehensive testing**: Added extensive test suite covering all supported formats with manual inspection output
+* **Comprehensive testing**: Added extensive test suite covering all supported formats including GIF with test fixtures
 
 # codecheck 0.22.0