transformers - ✅(Solved) Fix AutoConfig.register() ignored when trust_remote_code=True and auto_map is present [1 pull requests, 3 comments, 3 participants]

transformers2026-03-29 04:06:21

ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

GitHub issue URL

Helpful · Quick feedback

GitHub stats

huggingface/transformers#45093•Fetched 2026-04-08 01:45:14

View on GitHub

Comments

Participants

Timeline

Reactions

Author

Participants

Timeline (top)

referenced ×8commented ×3cross-referenced ×3closed ×2

AutoConfig.register() is silently ignored when trust_remote_code=True and the model's config.json contains auto_map.AutoConfig. This makes it impossible for downstream libraries to override a broken remote config class.

Root Cause

In configuration_auto.py, from_pretrained checks has_remote_code and trust_remote_code before checking CONFIG_MAPPING:

# Line 1478-1483
if has_remote_code and trust_remote_code:
    config_class = get_class_from_dynamic_module(class_ref, ...)  # always wins
    return config_class.from_pretrained(...)
elif "model_type" in config_dict:
    config_class = CONFIG_MAPPING[config_dict["model_type"]]  # never reached

When a class has been explicitly registered via AutoConfig.register(), it should take precedence over auto_map remote code — the caller has explicitly said "use this class for this model_type."

Fix Action

Fixed

Fixed by PR: fix: prefer registered config over remote code in AutoConfig.from_pretrained (https://github.com/huggingface/transformers/pull/45094)

PR fix notes

PR #45094: fix: prefer registered config over remote code in AutoConfig.from_pretrained

Repository: huggingface/transformers
Author: HanFa
State: open | merged: False
Link: https://github.com/huggingface/transformers/pull/45094

Description (problem / solution / changelog)

When a config class has been explicitly registered via AutoConfig.register(), it should take precedence over auto_map remote code. Previously, trust_remote_code=True with auto_map.AutoConfig in config.json would always load remote code, ignoring the registration.

This caused issues for downstream libraries (e.g., vLLM) that vendor fixed config classes for models with broken remote code — internal calls from AutoTokenizer/AutoProcessor would bypass the registration and load the broken remote class.

What does this PR do?

Fixes: https://github.com/huggingface/transformers/issues/45093

Code Agent Policy

The Transformers repo is currently being overwhelmed by a large number of PRs and issue comments written by code agents. We are currently bottlenecked by our ability to review and respond to them. As a result, we ask that new users do not submit pure code agent PRs at this time. You may use code agents in drafting or to help you diagnose issues. We'd also ask autonomous "OpenClaw"-like agents not to open any PRs or issues for the moment.

PRs that appear to be fully agent-written will probably be closed without review, and we may block users who do this repeatedly or maliciously.

This is a rapidly-evolving situation that's causing significant shockwaves in the open-source community. As a result, this policy is likely to be updated regularly in the near future. For more information, please read CONTRIBUTING.md.

I confirm that this is not a pure code agent PR.

Before submitting

This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
Did you read the contributor guideline, Pull Request section?
Was this discussed/approved via a Github issue or the forum? Please add a link to it if that's the case.
Did you make sure to update the documentation with your changes? Here are the documentation guidelines, and here are tips on formatting docstrings.
Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag members/contributors who may be interested in your PR.

Changed files

src/transformers/models/auto/auto_factory.py (modified, +2/-2)
src/transformers/models/auto/configuration_auto.py (modified, +1/-1)
src/transformers/models/auto/feature_extraction_auto.py (modified, +1/-1)
src/transformers/models/auto/image_processing_auto.py (modified, +1/-1)
src/transformers/models/auto/processing_auto.py (modified, +1/-1)
src/transformers/models/auto/tokenization_auto.py (modified, +1/-1)
src/transformers/models/auto/video_processing_auto.py (modified, +1/-1)
tests/models/auto/test_configuration_auto.py (modified, +6/-1)
tests/models/auto/test_feature_extraction_auto.py (modified, +10/-1)
tests/models/auto/test_image_processing_auto.py (modified, +10/-1)
tests/models/auto/test_modeling_auto.py (modified, +7/-1)
tests/models/auto/test_processor_auto.py (modified, +4/-4)
tests/models/auto/test_tokenization_auto.py (modified, +11/-0)
tests/models/auto/test_video_processing_auto.py (modified, +10/-1)

Code Example

from transformers import AutoConfig, PretrainedConfig

# 1. Register a local config class for a model_type
class MyFixedConfig(PretrainedConfig):
    model_type = "hyperclovax_vlm"
    def __init__(self, text_config=None, **kwargs):
        self.text_config = None  # always set
        super().__init__(**kwargs)
    def get_text_config(self, decoder=False):
        return self.text_config if self.text_config is not None else self

AutoConfig.register("hyperclovax_vlm", MyFixedConfig, exist_ok=True)

# 2. Load with trust_remote_code=False — uses registered class ✓
config = AutoConfig.from_pretrained(
    "naver-hyperclovax/HyperCLOVAX-SEED-Vision-Instruct-3B",
    trust_remote_code=False,
)
print(type(config))  # <class 'MyFixedConfig'>

# 3. Load with trust_remote_code=True — ignores registered class ✗
config = AutoConfig.from_pretrained(
    "naver-hyperclovax/HyperCLOVAX-SEED-Vision-Instruct-3B",
    trust_remote_code=True,
)
print(type(config))  # remote HCXVisionConfig, NOT MyFixedConfig
# This crashes because the remote code has a bug in get_text_config()

---

# Line 1478-1483
if has_remote_code and trust_remote_code:
    config_class = get_class_from_dynamic_module(class_ref, ...)  # always wins
    return config_class.from_pretrained(...)
elif "model_type" in config_dict:
    config_class = CONFIG_MAPPING[config_dict["model_type"]]  # never reached

---

vLLM code (we control):
    AutoProcessor.from_pretrained(..., trust_remote_code=True)  ← must pass True
                                                                   because the PROCESSOR
                                                                   itself is remote code

    transformers internals (we don't control):
      → AutoConfig.from_pretrained(..., trust_remote_code=True)  ✗ loads broken remote config

---

if has_remote_code and trust_remote_code and not has_local_code:
    config_class = get_class_from_dynamic_module(class_ref, ...)
elif "model_type" in config_dict:
    config_class = CONFIG_MAPPING[config_dict["model_type"]]

RAW_BUFFERClick to expand / collapse

Description

Reproduction

from transformers import AutoConfig, PretrainedConfig

# 1. Register a local config class for a model_type
class MyFixedConfig(PretrainedConfig):
    model_type = "hyperclovax_vlm"
    def __init__(self, text_config=None, **kwargs):
        self.text_config = None  # always set
        super().__init__(**kwargs)
    def get_text_config(self, decoder=False):
        return self.text_config if self.text_config is not None else self

AutoConfig.register("hyperclovax_vlm", MyFixedConfig, exist_ok=True)

# 2. Load with trust_remote_code=False — uses registered class ✓
config = AutoConfig.from_pretrained(
    "naver-hyperclovax/HyperCLOVAX-SEED-Vision-Instruct-3B",
    trust_remote_code=False,
)
print(type(config))  # <class 'MyFixedConfig'>

# 3. Load with trust_remote_code=True — ignores registered class ✗
config = AutoConfig.from_pretrained(
    "naver-hyperclovax/HyperCLOVAX-SEED-Vision-Instruct-3B",
    trust_remote_code=True,
)
print(type(config))  # remote HCXVisionConfig, NOT MyFixedConfig
# This crashes because the remote code has a bug in get_text_config()

Root Cause

In configuration_auto.py, from_pretrained checks has_remote_code and trust_remote_code before checking CONFIG_MAPPING:

# Line 1478-1483
if has_remote_code and trust_remote_code:
    config_class = get_class_from_dynamic_module(class_ref, ...)  # always wins
    return config_class.from_pretrained(...)
elif "model_type" in config_dict:
    config_class = CONFIG_MAPPING[config_dict["model_type"]]  # never reached

When a class has been explicitly registered via AutoConfig.register(), it should take precedence over auto_map remote code — the caller has explicitly said "use this class for this model_type."

Impact

This affects any downstream library (e.g., vLLM) that vendors config fixes for models with broken remote code. Even after registering the fixed config, internal calls from AutoTokenizer.from_pretrained() and AutoProcessor.from_pretrained() pass trust_remote_code=True and bypass the registration.

The code trace looks something like:

vLLM code (we control):
    AutoProcessor.from_pretrained(..., trust_remote_code=True)  ← must pass True
                                                                   because the PROCESSOR
                                                                   itself is remote code

    transformers internals (we don't control):
      → AutoConfig.from_pretrained(..., trust_remote_code=True)  ✗ loads broken remote config

Concrete example: vLLM vendors a fixed HCXVisionConfig to handle empty initialization (needed for v5's @strict validation). get_config() correctly uses the vendored class, but AutoProcessor internally calls AutoConfig.from_pretrained(trust_remote_code=True) which loads the broken remote code and crashes.

Related: vllm-project/vllm#38387, #44956

Suggested Fix

Only prefer remote code when no local/registered class exists:

if has_remote_code and trust_remote_code and not has_local_code:
    config_class = get_class_from_dynamic_module(class_ref, ...)
elif "model_type" in config_dict:
    config_class = CONFIG_MAPPING[config_dict["model_type"]]

Environment

transformers: main branch (commit 9a9997fd73)
Python: 3.12

extent analysis

Fix Plan

To fix the issue, we need to modify the from_pretrained method in configuration_auto.py to prefer registered local classes over remote code when trust_remote_code=True.

Here are the steps:

Modify the from_pretrained method to check for local classes before loading remote code.
Update the condition to load remote code only when no local class is registered.

Example code:

if has_remote_code and trust_remote_code and not CONFIG_MAPPING.get(config_dict["model_type"]):
    config_class = get_class_from_dynamic_module(class_ref, ...)  # load remote code
elif "model_type" in config_dict:
    config_class = CONFIG_MAPPING[config_dict["model_type"]]  # use registered local class

Alternatively, you can use the suggested fix:

if has_remote_code and trust_remote_code and not has_local_code:
    config_class = get_class_from_dynamic_module(class_ref, ...)
elif "model_type" in config_dict:
    config_class = CONFIG_MAPPING[config_dict["model_type"]]

Make sure to update the has_local_code variable to check if a local class is registered for the given model_type.

Verification

To verify the fix, run the reproduction code with trust_remote_code=True and check if the registered local class is used:

config = AutoConfig.from_pretrained(
    "naver-hyperclovax/HyperCLOVAX-SEED-Vision-Instruct-3B",
    trust_remote_code=True,
)
print(type(config))  # should print <class 'MyFixedConfig'>

Extra Tips

Make sure to test the fix with different models and configurations to ensure it works as expected.
Consider adding a test case to the transformers library to prevent similar issues in the future.
If you're using a downstream library like vLLM, make sure to update the library to use the fixed AutoConfig class.

Vote matrix · Quick signals

Works

Did the solution work? Tap to confirm.

Easy Fix

Was it a quick fix?

Time Saver

Did it save you time?

Blocking

Was it severely blocking?

Common Issue

Are others likely hitting this too?

Flaky / Intermittent

Is it intermittent?

Verified / Reproducible

Can you reproduce it reliably?

#conversation history #tool integration #LLM response #prompt template #agent execution

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Data

Security

Network

Code

UI/UX

Text

System

Multimedia

Protocol

API

Engineering

transformers - ✅(Solved) Fix AutoConfig.register() ignored when trust_remote_code=True and auto_map is present [1 pull requests, 3 comments, 3 participants]

Recommended Tools

GitHub issue graph ai analysis

Root Cause

Fix Action

Fixed

PR fix notes

PR #45094: fix: prefer registered config over remote code in AutoConfig.from_pretrained

Description (problem / solution / changelog)

What does this PR do?

Code Agent Policy

Before submitting

Who can review?

Changed files

Code Example

Description

Reproduction

Root Cause

Impact

Suggested Fix

Environment

extent analysis

Fix Plan

Verification

Extra Tips

Still need to ship something?

TRENDING

transformers - ✅(Solved) Fix AutoConfig.register() ignored when trust_remote_code=True and auto_map is present [1 pull requests, 3 comments, 3 participants]

Recommended Tools

GitHub issue graph ai analysis

Root Cause

Fix Action

Fixed

PR fix notes

PR #45094: fix: prefer registered config over remote code in AutoConfig.from_pretrained

Description (problem / solution / changelog)

What does this PR do?

Code Agent Policy

Before submitting

Who can review?

Changed files

Code Example

Description

Reproduction

Root Cause

Impact

Suggested Fix

Environment

extent analysis

Fix Plan

Verification

Extra Tips

Still need to ship something?

RELATED_DISCOVERY

TRENDING