Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wip: massive refactor of collection storage and layout #32

Draft
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

wllfaria
Copy link
Owner

Motivation

Currently, collections are stored as a tree, which looks very intuitive as a json of requests can easily be translated into a tree. But this added a lot of complexity when rendering and mutating objects, often having to traverse the tree and having to Rc or Arc everything.

This refactor intends to change all of that by reading a collection as separate slabs, which are vector-esque data structured that also provide access by a reliable key.

This basically touches everything inside of the project, as the tree is used by basically everything, but I do believe this is a change for good and that it will greatly improve the quality of the project.

The idea is that we will load the tree of requests and flatten it into a more linear structure whilst maintaining the hierarchy of folders and requests.

This PR will also be changing a lot of the current layout, as some stuff don't really make me happy as they are.

@wllfaria wllfaria added the enhancement New feature or request label Sep 17, 2024
@wllfaria wllfaria self-assigned this Sep 17, 2024
@wllfaria wllfaria force-pushed the main branch 2 times, most recently from 07bacb0 to 92413dd Compare October 2, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant