Skip to content

Feat/multi issue fix 1980 1869 1879 2027#2032

Open
singhanurag0317-bit wants to merge 4 commits into
utksh1:mainfrom
singhanurag0317-bit:feat/multi-issue-fix-1980-1869-1879-2027
Open

Feat/multi issue fix 1980 1869 1879 2027#2032
singhanurag0317-bit wants to merge 4 commits into
utksh1:mainfrom
singhanurag0317-bit:feat/multi-issue-fix-1980-1869-1879-2027

Conversation

@singhanurag0317-bit

Copy link
Copy Markdown

Description

Related Issues

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

…rflow

- Add line-clamp-2 to Toolkit.tsx tool name <h3> element
- Add line-clamp-2 to availability guidance <div> element
- Prevents long tool names from breaking card layout
- Add SandboxConfig to PluginMetadata model
- Implement --network none for Docker sandbox mode
- Implement unshare --net wrapper for non-Docker Linux
- Add tests for network namespace enforcement
- Log warning when network isolation requested on unsupported OS
- Add owner_id and shared columns to saved_views table
- Scope CRUD operations to view owner
- Allow shared views to be read-only for non-owners
- Add ownership-scoped duplicate name validation
- Recreate table to replace global UNIQUE constraint with per-owner uniqueness
- Add comprehensive ownership tests
- Add test for bulkDeleteTasks failure showing error toast
- Add test verifying selection state persists after failure
- Fix button selector to match actual UI text (Prune_Selected)
- Fix checkbox selector to use material icon divs
@utksh1 utksh1 added level:advanced 55 pts difficulty label for advanced contributor PRs type:feature Feature work category bonus label type:bug Bug fix work category bonus label area:frontend Frontend React/UI work area:backend Backend API, database, or service work area:plugins Scanner plugin metadata, schemas, or plugin runtime work labels Jul 24, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This combines four unrelated issue fixes across frontend, migrations, sandbox execution, and tests. Please split this into one PR per issue, resolve the merge conflict, and fix the backend-unit failure.

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a multi-issue branch that mixes executor, migrations, sandboxing, saved views, Toolkit, Scans tests, and unrelated fixes. Please split it into focused PRs that each close one approved issue, then rebase and fix backend-unit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:backend Backend API, database, or service work area:frontend Frontend React/UI work area:plugins Scanner plugin metadata, schemas, or plugin runtime work level:advanced 55 pts difficulty label for advanced contributor PRs type:bug Bug fix work category bonus label type:feature Feature work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants