feat: add log attr

[94noni]

👋🏻
according to https://github.com/php/frankenphp/blob/68573a9b640a679cbf420169cdbb960317c6db16/scaling.go#L142
some log may have attributes

---

I would like to display them on the ember tui, to have a direct vision of such data

---

ℹ️ untested for now, just opening as reflexion/discussion

[alexandre-daubois]

Thank you for proposing!

Good idea on principle: runtime logs carry useful diagnostic info we currently drop. My only doubt is the UX: appending [k=v k=v] inside an already-truncated message column will get noisy fast on logs with 4+ attributes. Worth considering a detail pane on row selection instead, keeping the inline suffix only as a compact fallback maybe!

[94noni]

that is an option indeed, a right panel on log line detail
I will think about it :)
thx for stoping by

[94noni]

@alexandre-daubois I took some time to redo/rewrite this PR (bootstrapped with ia a bit)
now there is a panel registered on logs (not routes aggregations) and displaying those attributes metadata, to avoid having to display those kv pairs on the default table, keeping it clean by default

would this be ok for you? so I can polish/finish this behavior
thx

[94noni]

probably a miss rewrite from my original makefile ;)

[94noni]

routesViews are aggregated so no unitary metadata

[alexandre-daubois]

would this be ok for you? so I can polish/finish this behavior

Let's go for it! Tell me when it's ready for review 🙂