Skip to content

[Chore] Merge Argilla v2.7.1#72

Merged
JonnyTran merged 107 commits into
developfrom
merge/argilla-v2.7.1
Jun 5, 2025
Merged

[Chore] Merge Argilla v2.7.1#72
JonnyTran merged 107 commits into
developfrom
merge/argilla-v2.7.1

Conversation

@JonnyTran

Copy link
Copy Markdown
Member

Description

This PR integrates Argilla v2.7.0 and v2.7.1 into Extralit while carefully preserving all Extralit-specific features and customizations. The merge maintains our document extraction functionality, table handling, and schema management while bringing in Argilla's new features.

Related Tickets & Documents

Closes #64

This pull request introduces several changes across the Argilla project, including updates to workflows, frontend components, server schemas, and search engine logic. The most significant changes focus on enhancing functionality, improving maintainability, and fixing inconsistencies. Below is a categorized summary of the most important changes:

Frontend Enhancements

Server Schema Improvements

  • argilla-server/src/argilla_server/api/schemas/v1/chat.py: Made message and role length configurable via environment variables for greater flexibility.
  • argilla-server/src/argilla_server/api/schemas/v1/users.py & argilla-server/src/argilla_server/api/schemas/v1/workspaces.py: Added support for optional predefined IDs when creating users and workspaces. [1] [2]

Search Engine Logic Refinements

Documentation and Changelog Updates

  • argilla-server/CHANGELOG.md: Documented new features, including predefined IDs for users and workspaces, and fixed issues related to indexing records and custom fields. [1] [2]

These changes collectively enhance the Argilla project by improving user experience, adding new features, and refining code quality.

leiyre and others added 30 commits October 29, 2024 11:08
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes #<issue_number>

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the 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)
- Refactor (change restructuring the codebase without changing
functionality)
- Improvement (change adding some improvement to an existing
functionality)
- Documentation update

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
…ating `dataset.last_activity_at` column (#5656)

# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes #<issue_number>

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Corrected "labelled" to "labeled" in [README.md].

This pull request addresses a minor typo found in repository. The typo
has been corrected to improve clarity and maintain the quality of the
documentation.

This change is purely cosmetic and does not affect functionality.

# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes #<issue_number>

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the 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)
- Refactor (change restructuring the codebase without changing
functionality)
- Improvement (change adding some improvement to an existing
functionality)
- Documentation update

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes #<issue_number>

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the 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)
- Refactor (change restructuring the codebase without changing
functionality)
- Improvement (change adding some improvement to an existing
functionality)
- Documentation update

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

- [x] Use the newer version of python 
- [x] Use the latest version of core dependencies. (without code
changes)
- [x] Upgrade docker images
- [x] Remove unnecessary dependencies.

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Refactor (change restructuring the codebase without changing
functionality)
- Improvement (change adding some improvement to an existing
functionality)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
# Description
Updated some of the missing German translations + a few aria updates

**Type of change**
- Improvement (change adding some improvement to an existing
functionality)
-
**How Has This Been Tested**
Local Build/Firefox

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…#5661)

# Add a high-contrast theme & improvements for the forced-colors mode

- Invisible borders for key items to make them visible in forced colors
mode via `border: 1px solid var(--bg-opacity-1)`
- Set a button outline of 5px in forced colors to make selections
visible via `@media (forced-colors: active) `
- add a new high contrast theme to the theme.scss & to the settings menu
- all new colors in the new high-contrast theme should create enough
contrast for WCAG AAA
- replace some hard coded colors with the colors from the current theme
- add hexToCssHsl to the Color.ts and resolveCssVariable to the
RecordStatus.ts to use theme vars there too

The high contrast theme is currently basically the light theme with
stronger colors so that it complies with WCAG AAA

![high_contrast_theme](https://github.com/user-attachments/assets/2ba38c76-9982-452d-83f0-6190aa09f7d5)


In the forced color mode you can now see certain features that where not
visible before:

![forced_colors](https://github.com/user-attachments/assets/7648bcb6-9f21-40d6-96f1-5df68f6f37be)

Forced contrast mode depends on the implementation of the OS & browser
therefore the selected labels I cannot just highlight with a background
color and I did it with a thick border.

**Type of change**

- New feature (non-breaking change which adds functionality)

**How Has This Been Tested**
Local Build - Firefox Nightly + Chromium

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…lembic (#5672)

# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Alembic will use psycopy2 driver by default. We still need to install it
to avoid startup errors.

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
# Description
Corrected the spelling mistakes and increased the grammatical
continuity.


**Type of change**
These changes improve the quality of the documentations. They do not
affect the system in any way.

---------

Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes argilla-io/argilla#5627

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Improvement (change adding some improvement to an existing
functionality)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

This PR allows users to change all the settings before creating the
dataset, and metadata and vector, when the dataset is created.

I included the `remove` method instead of a new
`settings.add(override=True)` since this change requires a lot of
internal refactor with the current design.

An example of how to use could be to change inferred settings from hub
before create the final dataset

```python
settings = rg.Settings.from_hub("google/frames-benchmark")

settings.fields.remove("answer")

for field in settings.fields:
    if field.name.startswith("wiki"):
        settings.fields.remove(field)
        settings.metadata.add(rg.TermsMetadataProperty(field.name))

settings.questions.add(rg.TextQuestion(name="answer", title="Answer"))

dataset = rg.Dataset.from_hub("google/frames-benchmark", settings=settings)
```

Or adding new metadata or vector settings when the dataset is created:
```python
dataset = client.dataset("my-dataset")

dataset.metadata.add([rg.TermsMetadata(name="split")])
dataset.update() # this line sends the change to the argilla server

```

## Updated

You can remove a property by name:
```python
settings.fields.remove("text")
```

or by property instance
```python
for field in settings.fields:
   if field.name.startwith("wiki"):
      settings.fields.remove(field)
````

And you can override existing properties to change the property type,
using the new `settings.add` method:
```python
settings = Settings.from_hub(...)

# change some settings definitions before create the dataset
settings.add(rg.TermsMetadata("wikipedia_1")) # this will remove the existing wikipedia_1 property and will create a new terms metadata one
```

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Refactor (change restructuring the codebase without changing
functionality)
- Improvement (change adding some improvement to an existing
functionality)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: burtenshaw <ben@argilla.io>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Check the role and content properties to validate the chat field

---------

Co-authored-by: Francisco Aranda <francis@argilla.io>
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes argilla-io/argilla#5633

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)
- New feature (non-breaking change which adds functionality)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: José Francisco Calvo <jose@argilla.io>
Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…xternal URL (#5675)

# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

This PR fixes the `router.go` method and avoids calling `router.push`
when the external flag is True

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
This PR fixes the visualisation of highlighted text in text and chat
fields adding an id
- [x] Rating values validate min / max
- [x] Question name format and renaming
- [x] Minor style adjustments 
- [x] Show required / not required in the form preview
- [x] Redirect from /new/repoId to /repoId
- [x] Redirect to error page when repoId is invalid

---------

Co-authored-by: Francisco Aranda <francis@argilla.io>
Co-authored-by: Paco Aranda <frascuchon@gmail.com>
Co-authored-by: José Francisco Calvo <jose@argilla.io>
Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->



**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Documentation update

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes argilla-io/argilla#5664

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Improvement (change adding some improvement to an existing
functionality)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: José Francisco Calvo <jose@argilla.io>
Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
fix #5659

---------

Co-authored-by: Damián Pumar <damianpumar@gmail.com>
# Description

This is the feature branch for Webhooks implementation.

Closes #1836 

**Type of change**

- New feature (non-breaking change which adds functionality)

**How Has This Been Tested**

- [ ] Manually tested locally and in HF spaces.

**Checklist**

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: Paco Aranda <frascuchon@gmail.com>
Co-authored-by: Damián Pumar <damianpumar@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Francisco Aranda <francis@argilla.io>
Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
Saikiranbonu1661 and others added 8 commits January 20, 2025 14:34
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes #5777 

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Improvement (change adding some improvement to an existing
functionality)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->
Passes similar param in query method while fetching records and could
see returning similarity score.

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: Sai Kiran Bonu <saikiran.bonu@phenompeople.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Francisco Aranda <francis@argilla.io>
Co-authored-by: Paco Aranda <frascuchon@gmail.com>
… (#5786)

# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

This PR allows users and workspaces to be created with predefined IDs.
Passing a predefined ID when creating workspaces and users simplifies
the migration process since users don't need to review responses and map
them to new users, avoiding potential issues in the process.


**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)
- Improvement (change adding some improvement to an existing
functionality)
- Documentation update

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

This PR fixes error when indexing records with missing values for chat
fields. Noticed in [this discord
thread](https://discord.com/channels/879548962464493619/1325817229954252841)

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
…rgilla client (#5789)

# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes #5548

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)
- Improvement (change adding some improvement to an existing
functionality)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

Closes argilla-io/argilla#5803

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->

- Bug fix (non-breaking change which fixes an issue)

**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)
Copilot AI review requested due to automatic review settings June 4, 2025 22:34
@JonnyTran JonnyTran requested review from a team as code owners June 4, 2025 22:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR merges Argilla v2.7.1 into Extralit while integrating new Argilla features and ensuring compatibility with existing customizations. Key changes include frontend UI enhancements, expanded API support for predefined IDs with additional tests, and server-side improvements such as configurable chat schema limits and refined search engine mapping logic.

Reviewed Changes

Copilot reviewed 52 out of 52 changed files in this pull request and generated no comments.

Show a summary per file
File Description
argilla-server/tests/unit/conftest.py Removed an unused import from the contexts module.
argilla-server/tests/unit/api/handlers/v1/workspaces.py Added tests covering workspace creation scenarios including predefined, missing, wrong, and duplicate IDs.
argilla-server/tests/unit/api/handlers/v1/users/test_create_user.py Added tests for user creation handling predefined, missing, wrong, and duplicate IDs.
argilla-server/src/argilla_server/search_engine/commons.py Updated license comment, adjusted mapping logic, and cleaned up abstract methods.
argilla-server/src/argilla_server/contexts/datasets.py Removed unused imports to streamline context functionality.
argilla-server/src/argilla_server/contexts/accounts.py Enhanced workspace and user creation to support optional IDs with uniqueness checks.
argilla-server/src/argilla_server/api/schemas/v1/workspaces.py Added an optional id field; note the inclusion of an unexpected import from optparse.
argilla-server/src/argilla_server/api/schemas/v1/users.py Added an optional id field for user creation schema.
argilla-server/src/argilla_server/api/schemas/v1/questions.py Removed obsolete comment lines.
argilla-server/src/argilla_server/api/schemas/v1/chat.py Made message and role length limits configurable via environment variables.
argilla-server/src/argilla_server/api/handlers/v1/workspaces.py Reformatting error responses for workspace deletion.
argilla-server/CHANGELOG.md & argilla-frontend/CHANGELOG.md Updated changelogs to document new features and fixes.
argilla-frontend/translation/* Added translations for exporting datasets to Hugging Face Hub in English and Spanish.
argilla-frontend/nuxt.config.ts Configured PostCSS with cssnano for production builds.
argilla-frontend/components/features/annotation/header/header-bar/ImportData.vue Updated background color for improved UI contrast.
.github/workflows/argilla.yml Changed workflow fail-fast setting from true to false.
Comments suppressed due to low confidence (2)

argilla-server/src/argilla_server/search_engine/commons.py:4

  • There's a TODO comment regarding an inconsistent license. Please resolve this discrepancy by updating or removing the license comment in accordance with the project's standard license.
# TODO: This license is not consistent with the license used in the project.

argilla-server/src/argilla_server/api/schemas/v1/workspaces.py:16

  • The import 'Option' from 'optparse' appears to be unused. Consider removing it to avoid confusion and maintain cleaner code.
from optparse import Option

@JonnyTran JonnyTran self-assigned this Jun 4, 2025
@JonnyTran JonnyTran added enhancement New feature or request refactor Code refactoring or technical debt improvements labels Jun 4, 2025
@JonnyTran JonnyTran added this to the v0.5.0 milestone Jun 4, 2025
@codecov

codecov Bot commented Jun 4, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 95.83333% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...r/src/argilla_server/api/handlers/v1/workspaces.py 0.00% 1 Missing ⚠️
Flag Coverage Δ
argilla-server 87.14% <95.83%> (+0.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...a-server/src/argilla_server/api/schemas/v1/chat.py 100.00% <100.00%> (ø)
...ver/src/argilla_server/api/schemas/v1/questions.py 98.36% <ø> (ø)
...-server/src/argilla_server/api/schemas/v1/users.py 100.00% <100.00%> (ø)
...er/src/argilla_server/api/schemas/v1/workspaces.py 100.00% <100.00%> (ø)
...lla-server/src/argilla_server/contexts/accounts.py 91.08% <100.00%> (+0.56%) ⬆️
...lla-server/src/argilla_server/contexts/datasets.py 91.11% <100.00%> (ø)
...server/src/argilla_server/search_engine/commons.py 92.56% <100.00%> (+1.97%) ⬆️
...r/src/argilla_server/api/handlers/v1/workspaces.py 75.60% <0.00%> (+9.75%) ⬆️

... and 3 files with indirect coverage changes

Components Coverage Δ
extralit ∅ <ø> (∅)
argilla ∅ <ø> (∅)
argilla_v1 ∅ <ø> (∅)
argilla-server 87.14% <95.83%> (+0.36%) ⬆️
argilla-frontend ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread .github/workflows/argilla-v1.yml Dismissed
@JonnyTran JonnyTran merged commit 9236786 into develop Jun 5, 2025
8 of 11 checks passed
jonnywireless pushed a commit that referenced this pull request Jul 31, 2025
* fix and update test snaps

* clean code

* chore: Update develop version

* [CI] Configure workflow permissions (#5648)

* fix: enable fail-fast strategy in CI workflow

* fix: Remove PostCSS config from Nuxt and update imports and license headers

* fix: downgrade setup-miniconda to v2 and change miniforge variant to Mambaforge

---------

Co-authored-by: Leire Aguirre <leire@argilla.io>
Co-authored-by: Francisco Aranda <francis@argilla.io>
Co-authored-by: Paco Aranda <frascuchon@gmail.com>
Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Co-authored-by: Rahul <rahulk_c21124@students.isquareit.edu.in>
Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
Co-authored-by: Farookh Zaheer Siddiqui <129654632+FarukhS52@users.noreply.github.com>
Co-authored-by: Paul Bauriegel <paul.bauriegel@web.de>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Aditya Kale <113056317+kaleaditya779@users.noreply.github.com>
Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
Co-authored-by: burtenshaw <ben@argilla.io>
Co-authored-by: José Francisco Calvo <jose@argilla.io>
Co-authored-by: Damián Pumar <damianpumar@gmail.com>
Co-authored-by: Natalia Elvira <126158523+nataliaElv@users.noreply.github.com>
Co-authored-by: sean-hickey-wf <102522212+sean-hickey-wf@users.noreply.github.com>
Co-authored-by: Paco Aranda <francisco.aranda@huggingface.co>
Co-authored-by: Hamel Husain <hamel.husain@gmail.com>
Co-authored-by: Louis Brulé Naudet <louisbrulenaudet@icloud.com>
Co-authored-by: Saikiranbonu1661 <141391289+Saikiranbonu1661@users.noreply.github.com>
Co-authored-by: Sai Kiran Bonu <saikiran.bonu@phenompeople.com>
JonnyTran added a commit that referenced this pull request Jul 31, 2025
* fix and update test snaps

* clean code

* chore: Update develop version

* [CI] Configure workflow permissions (#5648)

* fix: enable fail-fast strategy in CI workflow

* fix: Remove PostCSS config from Nuxt and update imports and license headers

* fix: downgrade setup-miniconda to v2 and change miniforge variant to Mambaforge

---------

Co-authored-by: Leire Aguirre <leire@argilla.io>
Co-authored-by: Francisco Aranda <francis@argilla.io>
Co-authored-by: Paco Aranda <frascuchon@gmail.com>
Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Co-authored-by: Rahul <rahulk_c21124@students.isquareit.edu.in>
Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
Co-authored-by: Farookh Zaheer Siddiqui <129654632+FarukhS52@users.noreply.github.com>
Co-authored-by: Paul Bauriegel <paul.bauriegel@web.de>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Aditya Kale <113056317+kaleaditya779@users.noreply.github.com>
Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
Co-authored-by: burtenshaw <ben@argilla.io>
Co-authored-by: José Francisco Calvo <jose@argilla.io>
Co-authored-by: Damián Pumar <damianpumar@gmail.com>
Co-authored-by: Natalia Elvira <126158523+nataliaElv@users.noreply.github.com>
Co-authored-by: sean-hickey-wf <102522212+sean-hickey-wf@users.noreply.github.com>
Co-authored-by: Paco Aranda <francisco.aranda@huggingface.co>
Co-authored-by: Hamel Husain <hamel.husain@gmail.com>
Co-authored-by: Louis Brulé Naudet <louisbrulenaudet@icloud.com>
Co-authored-by: Saikiranbonu1661 <141391289+Saikiranbonu1661@users.noreply.github.com>
Co-authored-by: Sai Kiran Bonu <saikiran.bonu@phenompeople.com>
jonnywireless pushed a commit that referenced this pull request Aug 2, 2025
* fix and update test snaps

* clean code

* chore: Update develop version

* [CI] Configure workflow permissions (#5648)

* fix: enable fail-fast strategy in CI workflow

* fix: Remove PostCSS config from Nuxt and update imports and license headers

* fix: downgrade setup-miniconda to v2 and change miniforge variant to Mambaforge

---------

Co-authored-by: Leire Aguirre <leire@argilla.io>
Co-authored-by: Francisco Aranda <francis@argilla.io>
Co-authored-by: Paco Aranda <frascuchon@gmail.com>
Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Co-authored-by: Rahul <rahulk_c21124@students.isquareit.edu.in>
Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
Co-authored-by: Farookh Zaheer Siddiqui <129654632+FarukhS52@users.noreply.github.com>
Co-authored-by: Paul Bauriegel <paul.bauriegel@web.de>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Aditya Kale <113056317+kaleaditya779@users.noreply.github.com>
Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
Co-authored-by: burtenshaw <ben@argilla.io>
Co-authored-by: José Francisco Calvo <jose@argilla.io>
Co-authored-by: Damián Pumar <damianpumar@gmail.com>
Co-authored-by: Natalia Elvira <126158523+nataliaElv@users.noreply.github.com>
Co-authored-by: sean-hickey-wf <102522212+sean-hickey-wf@users.noreply.github.com>
Co-authored-by: Paco Aranda <francisco.aranda@huggingface.co>
Co-authored-by: Hamel Husain <hamel.husain@gmail.com>
Co-authored-by: Louis Brulé Naudet <louisbrulenaudet@icloud.com>
Co-authored-by: Saikiranbonu1661 <141391289+Saikiranbonu1661@users.noreply.github.com>
Co-authored-by: Sai Kiran Bonu <saikiran.bonu@phenompeople.com>
jonnywireless pushed a commit that referenced this pull request Aug 2, 2025
* fix and update test snaps

* clean code

* chore: Update develop version

* [CI] Configure workflow permissions (#5648)

* fix: enable fail-fast strategy in CI workflow

* fix: Remove PostCSS config from Nuxt and update imports and license headers

* fix: downgrade setup-miniconda to v2 and change miniforge variant to Mambaforge

---------

Co-authored-by: Leire Aguirre <leire@argilla.io>
Co-authored-by: Francisco Aranda <francis@argilla.io>
Co-authored-by: Paco Aranda <frascuchon@gmail.com>
Co-authored-by: José Francisco Calvo <josefranciscocalvo@gmail.com>
Co-authored-by: Rahul <rahulk_c21124@students.isquareit.edu.in>
Co-authored-by: Sara Han <127759186+sdiazlor@users.noreply.github.com>
Co-authored-by: Farookh Zaheer Siddiqui <129654632+FarukhS52@users.noreply.github.com>
Co-authored-by: Paul Bauriegel <paul.bauriegel@web.de>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Aditya Kale <113056317+kaleaditya779@users.noreply.github.com>
Co-authored-by: David Berenstein <david.m.berenstein@gmail.com>
Co-authored-by: burtenshaw <ben@argilla.io>
Co-authored-by: José Francisco Calvo <jose@argilla.io>
Co-authored-by: Damián Pumar <damianpumar@gmail.com>
Co-authored-by: Natalia Elvira <126158523+nataliaElv@users.noreply.github.com>
Co-authored-by: sean-hickey-wf <102522212+sean-hickey-wf@users.noreply.github.com>
Co-authored-by: Paco Aranda <francisco.aranda@huggingface.co>
Co-authored-by: Hamel Husain <hamel.husain@gmail.com>
Co-authored-by: Louis Brulé Naudet <louisbrulenaudet@icloud.com>
Co-authored-by: Saikiranbonu1661 <141391289+Saikiranbonu1661@users.noreply.github.com>
Co-authored-by: Sai Kiran Bonu <saikiran.bonu@phenompeople.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request refactor Code refactoring or technical debt improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] Merge Argilla v2.7.1