Skip to content

refactor(artifacts): simplify artifact file handling and remove unuse… - #13

Merged
Manasa-uigraph merged 1 commit into
mainfrom
refactor-diagrams
Oct 21, 2026
Merged

Manasa-uigraph merged 1 commit into
mainfrom
refactor-diagrams

Conversation

@Manasa-uigraph

@Manasa-uigraph Manasa-uigraph commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

…d comment features

  • Updated build command in CONTRIBUTING.md to increase memory limit.
  • Refactored artifact file handling by removing unnecessary checks for artifact paths in delete operations across multiple files.
  • Introduced new tests for diagram image handling in diagram-images.test.ts.
  • Removed deprecated comment-related components and functionality from the viewer, including virtual cursor and comments context.
  • Adjusted diagram node handling to exclude comments from search indexing.

What and why

How to test

Checklist

  • pnpm typecheck && pnpm lint && pnpm build pass
  • Tests added or updated where behavior changed
  • Docs updated if user-facing behavior changed
  • Does not weaken a product guarantee in CONTRIBUTING.md (or the PR explains why)

…d comment features

- Updated build command in CONTRIBUTING.md to increase memory limit.
- Refactored artifact file handling by removing unnecessary checks for artifact paths in delete operations across multiple files.
- Introduced new tests for diagram image handling in diagram-images.test.ts.
- Removed deprecated comment-related components and functionality from the viewer, including virtual cursor and comments context.
- Adjusted diagram node handling to exclude comments from search indexing.
@Manasa-uigraph
Manasa-uigraph merged commit 5e28c82 into main Oct 21, 2026
1 check passed
@Manasa-uigraph
Manasa-uigraph deleted the refactor-diagrams branch October 21, 2026 19:33
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.

1 participant