Skip to content

Conversation

@krishna9358
Copy link
Contributor

@krishna9358 krishna9358 commented Jan 22, 2026

Summary

  1. Currently the artifact's row doesn't help. We should display the following:
  • Workflow name .
  • Link to run from which the artifact was generated.
  • Actions should have 'Download' and 'Delete' icon. and left align 'Actions' in the column.
  • Remove the 'Copy ID' button. serves no purpose.
  • Remove the 'COMPONENT' column.
  1. Artifact Download Button Fix (Backend)
    Issue: Download buttons in the Run Artifacts Panel weren't working.

Root Cause: The backend URL path used {artifactId} but the DTO schema expected
id, causing a parameter mismatch.

Files Changed:

backend/src/storage/dto/artifacts.dto.ts

  • Added new RunArtifactIdParamSchema and RunArtifactIdParamDto
    that uses artifactId as the property name
    backend/src/workflows/workflows.controller.ts
  • Updated to use the new DTO and fixed the import to remove unused types
    frontend/src/services/api.ts
  • Changed
    downloadArtifact
    to use direct fetch with manually constructed URL (bypassing the typed client with the OpenAPI spec bug)
  1. Blank Canvas When Navigating From Artifact Library (Frontend)
    Issue: Clicking run links in the Artifact Library opened the workflow page with a blank canvas; only refreshing showed the graph.

Root Cause: When navigating directly to a run URL, the execution graph wasn't being initialized because the code was skipping initialization when there was a run context.

Files Changed:

frontend/src/features/workflow-builder/WorkflowBuilder.tsx

  • Changed logic to always initialize execution nodes when they're empty, regardless of run context
    frontend/src/features/workflow-builder/hooks/useWorkflowExecutionLifecycle.ts
  • Added safety checks to avoid hydrating from empty design state
  1. Added delete functionality to delete the artifacts.

Testing

  • bun run test
  • bun run lint
  • bun run typecheck
  • Additional notes:

Documentation

  • Updated the relevant doc(s) (see docs/guide.md) or checked that no updates are needed.
  • Recorded contract/architecture changes in both public docs and .ai logs when applicable.

Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
Signed-off-by: Krishna Mohan <krishanmohank974@gmail.com>
@krishna9358 krishna9358 force-pushed the @krishna9358/artifect-fixes branch from ed8df46 to 734c820 Compare January 23, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants