Open Bug 1642461 Opened 4 years ago Updated 2 years ago

Format logged ClojureScript data structures

Categories

(DevTools :: Console, enhancement, P3)

enhancement

Tracking

(Not tracked)

People

(Reporter: Harald, Unassigned)

References

(Blocks 2 open bugs)

Details

Attachments

(1 obsolete file)

Chrome DevTools' Custom Formatters feature is not secure and seems to be only used by ClojureScript DevTools. We still want to unblock CLJS developers to have a better experience with DevTools, similar how we format framework stacks and identify frameworks for file icons.

The basic idea that we want to validate is implementing CLJS' formatters as DevTools reps.

I'd be happy to mentor anyone who wants to implement this.
The first thing to do would be to do an inventory of the different type of ClojureScript objects that can be rendered, so we can split that bug into multiple smaller ones for each type of object.

Mentor: nchevobbe

Types for immutable js: List, Map, Ordered Map, Seq, Record, Set, Ordered Set, Stack, Seq. Should be pretty similar in clojurescript

from mori (A library for using ClojureScript's persistent data structures in JS) the cljs data structures should be: list, vector, hashMap, set, sortedSet, range, queue

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #1)

I'd be happy to mentor anyone who wants to implement this.
The first thing to do would be to do an inventory of the different type of ClojureScript objects that can be rendered, so we can split that bug into multiple smaller ones for each type of object.

Sign me up! I’d love to help.

Sadly I won't have any time to dedicate to this

Mentor: nchevobbe

For some reason, it also seems that CLJS modified the devtoolsFormatters array
and sometimes it couldn't find the body hook to use based on the index.
So instead of sending the index, we store the formatter entry in the object actor,
so we can reuse it later when customFormatterBody is called.

Attachment #9313291 - Attachment description: WIP: Bug 1642461- Add onNativeCall extra allow list so CLJS devtools work → Bug 1642461- Add onNativeCall extra allow list so CLJS devtools work
Attachment #9313291 - Attachment description: Bug 1642461- Add onNativeCall extra allow list so CLJS devtools work → WIP: - Add onNativeCall extra allow list so CLJS devtools work
Attachment #9313291 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: