Allow for HF org with different name from code platform org by egrace479 · Pull Request #110 · Imageomics/catalog

egrace479 · 2026-06-11T21:42:16Z

As I was setting up a test config for a Codeberg organization, the challenge of a Hugging Face org having a different name than the code platform org became glaringly obvious. This PR fixes that by adding a HF_ORGANIZATION_NAME parameter. As discussed in #93, adding a different data or model platform would be a much bigger project requiring community motivated use-case(s), and having HF optional would be similar. Thus, this is a simple parameter addition; the app also requires it to be filled.

It also adds a config validation check on Vite start which includes a check that the organization name variables are follow platform format specifications. It is better, especially in a dev/setup context, to have the app fail with an informative message instead of appearing to work but with only online console errors. Example local console output with invalid HF org name and unsupported platform passed:

$terminal error message with red x [CONFIG ERROR] Vite failed to start due to the following configuration errors: \n 1. HF_ORGANIZATION_NAME (abc center) is invalid, only letters, numbers, hyphens, and underscores are allowed \n 2. PLATFORM must be one of: github \n Please fix the listed issue(s) in ./public/config.yaml and try again.$

The config path is defined from a variable in the message.

…bles

…es are lowercase avoid quiet fails

Copilot

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Copilot

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

beanbean9339 · 2026-06-13T18:46:12Z

-ORGANIZATION_NAME: imageomics       # GitHub/Hugging Face organization name (lowercase for API calls)
-ORG_NAME: Imageomics                 # Display name for GitHub organization (can differ from API name)
+ORGANIZATION_NAME: imageomics       # Codebase platform organization name (lowercase for API calls)
+HF_ORGANIZATION_NAME: imageomics    # Hugging Face organization name (lowercase for API calls)


Since HF_ORGANIZATION_NAME is now required, have we confirmed this won't break existing deployments using older config files that only define ORGANIZATION_NAME?

I can build in backwards compatibility by using ORGANIZATION_NAME as a fallback, but I've also added validation for the config earlier and am fixing some of the other fallbacks, so I'm not sure we'd want to keep pulling that forward.

beanbean9339

I think overall it looks good to go. Just a few things I noticed while reviewing, but none of them would block functionality, more so things to consider:

The lowercase validation is helpful, though it may be worth considering validation against allowed organization name characters in the future rather than only checking case.
CONFIG_PATHandparsedConfiginvite.config.jscould potentially beconstinstead oflet` since they don't appear to be reassigned.

Other than that, the implementation, tests, and documentation updates all look good to me

…ame formats

Copilot

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

no need to do more if not defined

Copilot

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Copilot

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

beanbean9339

Overall looks good to me. Nice work separating the HF org handling and tightening up validation and tests. Looks good to merge 👍

egrace479 added 3 commits June 11, 2026 17:02

Allow for HF org with different name from codebase platform

2b9eab7

Describe new HF Org name option

1427bce

Add rendered error message in case of missing organization name varia…

6e73349

…bles

egrace479 added enhancement New feature or request structure Refactoring or architecture, general code organization labels Jun 11, 2026

egrace479 requested a review from Copilot June 11, 2026 21:42

Copilot started reviewing on behalf of egrace479 June 11, 2026 21:42 View session

This comment was marked as outdated.

Sign in to view

egrace479 added 3 commits June 11, 2026 17:56

Formatting fixes requested by Copilot

65b7c57

Use HF org name variable for HF tag export

cdb7cbc

Add HF org name variable to validation

8289e8a

egrace479 requested a review from Copilot June 11, 2026 22:00

Copilot started reviewing on behalf of egrace479 June 11, 2026 22:00 View session

This comment was marked as outdated.

Sign in to view

egrace479 added 3 commits June 11, 2026 19:17

Remove extra UI errors

4a1a0ab

Add config validation on dev runs and check organization name variabl…

0d84a46

…es are lowercase avoid quiet fails

Fix test expectations, error includes a helpful message

093bd05

egrace479 force-pushed the feat/mismatched-hf branch from 324f8b8 to 093bd05 Compare June 11, 2026 23:36

egrace479 requested a review from Copilot June 11, 2026 23:36

Copilot started reviewing on behalf of egrace479 June 11, 2026 23:37 View session

Copilot AI reviewed Jun 11, 2026

View reviewed changes

Comment thread src/validateConfig.js Outdated

Comment thread package.json Outdated

Comment thread vite.config.js Outdated

improve error message handling and remove unneeded command

59553c4

egrace479 requested a review from Copilot June 11, 2026 23:56

Copilot started reviewing on behalf of egrace479 June 11, 2026 23:56 View session

Copilot AI reviewed Jun 11, 2026

View reviewed changes

Comment thread vite.config.js Outdated

Comment thread vite.config.js Outdated

Comment thread src/validateConfig.js

Comment thread src/validateConfig.js

Clarify validation and use config path variable for loading error

99fbc3d

egrace479 requested a review from beanbean9339 June 13, 2026 18:34

beanbean9339 reviewed Jun 13, 2026

View reviewed changes

Comment thread vite.config.js

beanbean9339 reviewed Jun 13, 2026

View reviewed changes

Comment thread src/validateConfig.js Outdated

beanbean9339 approved these changes Jun 13, 2026

View reviewed changes

Fix: HF API is case-sensitive, fallback used where stated; test org n…

df270ba

…ame formats

egrace479 requested a review from Copilot June 15, 2026 23:16

Copilot started reviewing on behalf of egrace479 June 15, 2026 23:16 View session

Copilot AI reviewed Jun 15, 2026

View reviewed changes

Comment thread src/validateConfig.js Outdated

Comment thread src/validateConfig.js Outdated

Comment thread main.js

egrace479 added 2 commits June 15, 2026 19:39

Move initialization guard to immediately after variables set

1fa1c62

no need to do more if not defined

fix HF regex and clarify code description

8d0432e

egrace479 requested a review from Copilot June 15, 2026 23:42

Copilot started reviewing on behalf of egrace479 June 15, 2026 23:42 View session

Copilot AI reviewed Jun 15, 2026

View reviewed changes

Comment thread vite.config.js

Comment thread src/validateConfig.js

Comment thread public/config.yaml

Comment thread docs/personalization.md

Comment thread AGENTS.md

Comment thread tests/validateConfig.test.js

egrace479 requested a review from Copilot June 16, 2026 00:14

Copilot started reviewing on behalf of egrace479 June 16, 2026 00:14 View session

Copilot AI reviewed Jun 16, 2026

View reviewed changes

Comment thread src/validateConfig.js Outdated

Validate PLATFORM variable type to provide formatted error

81a2373

egrace479 requested a review from beanbean9339 June 16, 2026 00:54

beanbean9339 approved these changes Jun 16, 2026

View reviewed changes

egrace479 merged commit ba4b755 into dev Jun 16, 2026
1 check passed

egrace479 deleted the feat/mismatched-hf branch June 16, 2026 14:02

Conversation

egrace479 commented Jun 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

beanbean9339 Jun 13, 2026

Choose a reason for hiding this comment

Uh oh!

egrace479 Jun 15, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

beanbean9339 left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

beanbean9339 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

egrace479 commented Jun 11, 2026 •

edited

Loading