Add file size column to workfiles in launcher

[mahesh-ynput]

Changelog Description

The Launcher currently shows no file metadata such as timestamp or file size. Could you please add file size and last modified timestamp allowing artists to quickly spot zero-size or potentially corrupt files before wasting time attempting to open them?

Additional info

last modified column is already there now implemented other column where includes workfile size

Testing notes:

1. Open launcher select project->folder->task
2. See workfiles size section 3rd column

[mahesh-ynput]

Resizing column is in pending.

[Copilot]

Pull request overview

Adds workfile file-size metadata to the Launcher “Workfiles” listing so artists can spot empty/corrupt files before opening them.

Changes:

• Extend WorkfileItem with an optional file_size field.
• Populate file_size in the launcher backend model when the workfile exists on disk.
• Add a new “Size” column in the workfiles UI model and display formatted sizes.

Reviewed changes

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

File	Description
client/ayon_core/tools/launcher/ui/workfiles_page.py	Adds a 3rd column (“Size”), stores FILE_SIZE_ROLE, formats size for display, and exposes it through the model.
client/ayon_core/tools/launcher/models/workfiles.py	Computes file size from disk for existing workfiles and passes it into WorkfileItem.
client/ayon_core/tools/launcher/abstract.py	Extends WorkfileItem dataclass with optional file_size.

Comments suppressed due to low confidence (1)

client/ayon_core/tools/launcher/ui/workfiles_page.py:266

• Size values are formatted to strings for DisplayRole, so sorting the "Size" column will be lexicographic (e.g. "900.0 KB" > "1.0 MB"). Add a numeric comparison for column 2 using FILE_SIZE_ROLE in the proxy's lessThan so sort works as expected.

class WorkfileSortFilterProxy(QtCore.QSortFilterProxyModel):
    def lessThan(self, source_left, source_right):
        l_host = source_left.data(HOST_NAME_ROLE)
        r_host = source_right.data(HOST_NAME_ROLE)
        if l_host != r_host:

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[BigRoy]

Can we also make sure that by default there's no horizontal scrollbar - so we'll need to pick some decent default widths? Or trigger resize to fit contents for the columns whenever we switch - but not sure that's entirely nice 🤔

[mahesh-ynput]

Can we also make sure that by default there's no horizontal scrollbar - so we'll need to pick some decent default widths? Or trigger resize to fit contents for the columns whenever we switch - but not sure that's entirely nice 🤔

22a8e4f

[BigRoy]

LGTM

[moonyuet]

Does testing with the launcher, working as expected. Please take a look at review and do the code changes so that we can possibly merge this PR. Thanks

[moonyuet]

See the suggestion below.

[moonyuet]

I did the test on the launcher with the updated commit. looks good.
Are we gonna merge this or we need to resolve something else?

[BigRoy]

I think we can merge it. Unless we want to keep ui-qt-luncher branch lean for the time being.

Will leave up to @iLLiCiTiT