codex - ✅(Solved) Fix Artifact feature seems to dead code? [1 pull requests, 1 comments, 2 participants]

Official PRs (…)
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.

Helpful · Quick feedback

Loading…
GitHub stats
openai/codex#17754Fetched 2026-04-15 06:28:56
View on GitHub
Comments
1
Participants
2
Timeline
4
Reactions
0
Author
Participants
Timeline (top)
labeled ×2closed ×1commented ×1

PR fix notes

PR #15851: feat: drop artifact tool and feature

Description (problem / solution / changelog)

(No description)

Changed files

  • codex-rs/Cargo.lock (modified, +0/-1)
  • codex-rs/Cargo.toml (modified, +1/-0)
  • codex-rs/core/Cargo.toml (modified, +0/-1)
  • codex-rs/core/config.schema.json (modified, +0/-6)
  • codex-rs/core/src/config/schema.rs (modified, +3/-0)
  • codex-rs/core/src/lib.rs (modified, +0/-1)
  • codex-rs/core/src/packages/mod.rs (removed, +0/-1)
  • codex-rs/core/src/packages/versions.rs (removed, +0/-2)
  • codex-rs/core/src/tools/handlers/artifacts.rs (removed, +0/-295)
  • codex-rs/core/src/tools/handlers/artifacts_tests.rs (removed, +0/-98)
  • codex-rs/core/src/tools/handlers/mod.rs (modified, +0/-2)
  • codex-rs/core/src/tools/spec.rs (modified, +0/-43)
  • codex-rs/core/src/tools/spec_tests.rs (modified, +0/-22)
  • codex-rs/core/templates/tools/presentation_artifact.md (removed, +0/-200)

Code Example

diff --git a/codex-rs/config/src/schema.rs b/codex-rs/config/src/schema.rs
index 91aa50add..3d908051d 100644
--- a/codex-rs/config/src/schema.rs
+++ b/codex-rs/config/src/schema.rs
@@ -22,9 +22,6 @@ pub fn features_schema(schema_gen: &mut SchemaGenerator) -> Schema {
 
     let mut validation = ObjectValidation::default();
     for feature in FEATURES {
-        if feature.id == codex_features::Feature::Artifact {
-            continue;
-        }
         if feature.id == codex_features::Feature::MultiAgentV2 {
             validation.properties.insert(
                 feature.key.to_string(),
diff --git a/codex-rs/features/src/lib.rs b/codex-rs/features/src/lib.rs
index 10d5fdf88..42e162236 100644
--- a/codex-rs/features/src/lib.rs
+++ b/codex-rs/features/src/lib.rs
@@ -170,8 +170,6 @@ pub enum Feature {
     ToolCallMcpElicitation,
     /// Enable personality selection in the TUI.
     Personality,
-    /// Enable native artifact tools.
-    Artifact,
     /// Enable Fast mode selection in the TUI and request layer.
     FastMode,
     /// Enable experimental realtime voice conversation mode in the TUI.
@@ -851,12 +849,6 @@ pub const FEATURES: &[FeatureSpec] = &[
         stage: Stage::Stable,
         default_enabled: true,
     },
-    FeatureSpec {
-        id: Feature::Artifact,
-        key: "artifact",
-        stage: Stage::UnderDevelopment,
-        default_enabled: false,
-    },
     FeatureSpec {
         id: Feature::FastMode,
         key: "fast_mode",
RAW_BUFFERClick to expand / collapse

It seems the aftifact feature is dead code.

This repository has been An owner of this repository has limited the ability to open a pull request to users that are collaborators on this repository., so just post an issue instead. If a code fix suggestion is unnecessary, even if it's in issue format, I will refrain from posting it in the future. Please point it out.


regression: https://github.com/openai/codex/pull/15851

diff --git a/codex-rs/config/src/schema.rs b/codex-rs/config/src/schema.rs
index 91aa50add..3d908051d 100644
--- a/codex-rs/config/src/schema.rs
+++ b/codex-rs/config/src/schema.rs
@@ -22,9 +22,6 @@ pub fn features_schema(schema_gen: &mut SchemaGenerator) -> Schema {
 
     let mut validation = ObjectValidation::default();
     for feature in FEATURES {
-        if feature.id == codex_features::Feature::Artifact {
-            continue;
-        }
         if feature.id == codex_features::Feature::MultiAgentV2 {
             validation.properties.insert(
                 feature.key.to_string(),
diff --git a/codex-rs/features/src/lib.rs b/codex-rs/features/src/lib.rs
index 10d5fdf88..42e162236 100644
--- a/codex-rs/features/src/lib.rs
+++ b/codex-rs/features/src/lib.rs
@@ -170,8 +170,6 @@ pub enum Feature {
     ToolCallMcpElicitation,
     /// Enable personality selection in the TUI.
     Personality,
-    /// Enable native artifact tools.
-    Artifact,
     /// Enable Fast mode selection in the TUI and request layer.
     FastMode,
     /// Enable experimental realtime voice conversation mode in the TUI.
@@ -851,12 +849,6 @@ pub const FEATURES: &[FeatureSpec] = &[
         stage: Stage::Stable,
         default_enabled: true,
     },
-    FeatureSpec {
-        id: Feature::Artifact,
-        key: "artifact",
-        stage: Stage::UnderDevelopment,
-        default_enabled: false,
-    },
     FeatureSpec {
         id: Feature::FastMode,
         key: "fast_mode",

extent analysis

TL;DR

The artifact feature appears to be removed, so any references to it should be updated or removed to resolve the issue.

Guidance

  • Review the codebase for any remaining references to the artifact feature and remove or update them as necessary.
  • Verify that the removal of the artifact feature does not introduce any regressions or break existing functionality.
  • Check the codex-rs/config/src/schema.rs and codex-rs/features/src/lib.rs files for any other features that may be affected by the removal of the artifact feature.
  • Consider updating the documentation to reflect the removal of the artifact feature.

Example

No code snippet is provided as the issue is more related to code removal rather than addition.

Notes

The removal of the artifact feature seems to be intentional, as indicated by the comment "It seems the aftifact feature is dead code." However, it's essential to ensure that this removal does not introduce any unintended consequences.

Recommendation

Apply workaround: Remove any references to the artifact feature, as it appears to be dead code and is no longer supported.

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?
Loading…

Still need to ship something?

×6

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

Back to top recommendations

TRENDING