Sam Denty

icon indicating a website link https://samdenty.com/ icon indicating an email [email protected]

icon company @vercel

Results 79 comments of Sam Denty

Set array items display incorrectly

I believe this stems from the underlying react-inspector library. With all the past issues opened though from styling/virtualization, I'm tempted more towards a rewrite. I saw that https://github.com/facebook/flipper started using...

Set array items display incorrectly

we'd need a new transform similar to the promise one https://github.com/samdenty/console-feed/issues/86. https://github.com/samdenty/console-feed/blob/0405ab2602c9eb14a4bac7499aa86189531827e6/src/Component/react-inspector/index.tsx#L86 though the promise state can't update due to lack of virtualization

console.log(event) doesn't print all the key value pairs

I believe the event properties are stored on prototype. We don't serialize prototypes currently as that can be quite expensive.

console.log(event) doesn't print all the key value pairs

you should fork - you could add a custom transform for Event (https://github.com/samdenty/console-feed/blob/master/src/Transform/index.ts), or change something here - https://github.com/samdenty/console-feed/blob/3fc8a958640f71d06d8edcdd1a00a300c4b9337c/src/Transform/replicator/index.ts#L97, probably remove the reflect.has check

console.log(event) doesn't print all the key value pairs

@chaitanya71998 hmm that's odd. You can try manually building with `tsc -p ./tsconfig.build.json --declaration`

Support for console.group

Not supported yet unfortunately. Don't have time to implement - but am open to accepting prs!

CSP (Content Security Policy) unsafe-eval

@franciscolourenco That wouldn't work for web workers etc, if you want to send a PR that uses globalThis go ahead. copied from https://github.com/inikulin/replicator

Changing object's property values during console log causes a bug

See https://github.com/samdenty/console-feed/issues/19#issuecomment-483432656

Code breaks for objects created with Object.create(null)

`replicator/index.ts` is forked from https://www.npmjs.com/package/replicator, probably best to send a pr there and merge back

[Feature Request] Persistence Module

Hey there, could you elaborate and provide relevant sources regarding `cash poisoning`? I presume you mean cache poisoning => https://github.com/beefproject/beef/wiki/Persistence Currently injectify has two levels of persistence - Core Cache...