Skip to content

Commit d3a96e9

Browse files
cschleidencschleiden
authored
Merge pull request #419 from cschleiden/copilot/fix-338
Improve JSON output formatting in web UI
2 parents 98500bc + 9f32e24 commit d3a96e9

11 files changed

+50
-16
lines changed

‎diag/app/build/asset-manifest.json‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
{
22
"files": {
33
"main.css": "./static/css/main.8b0db0ad.css",
4-
"main.js": "./static/js/main.e117a19e.js",
4+
"main.js": "./static/js/main.1d3b50fb.js",
55
"index.html": "./index.html",
66
"main.8b0db0ad.css.map": "./static/css/main.8b0db0ad.css.map",
7-
"main.e117a19e.js.map": "./static/js/main.e117a19e.js.map"
7+
"main.1d3b50fb.js.map": "./static/js/main.1d3b50fb.js.map"
88
},
99
"entrypoints": [
1010
"static/css/main.8b0db0ad.css",
11-
"static/js/main.e117a19e.js"
11+
"static/js/main.1d3b50fb.js"
1212
]
1313
}

‎diag/app/build/index.html‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
<!doctype html><html lang="en"><head><meta charset="utf-8"/><meta name="viewport" content="width=device-width,initial-scale=1"/><meta name="theme-color" content="#000000"/><title>go-workflows</title><script defer="defer" src="./static/js/main.e117a19e.js"></script><link href="./static/css/main.8b0db0ad.css" rel="stylesheet"></head><body><noscript>You need to enable JavaScript to run this app.</noscript><div id="root"></div></body></html>
1+
<!doctype html><html lang="en"><head><meta charset="utf-8"/><meta name="viewport" content="width=device-width,initial-scale=1"/><meta name="theme-color" content="#000000"/><title>go-workflows</title><script defer="defer" src="./static/js/main.1d3b50fb.js"></script><link href="./static/css/main.8b0db0ad.css" rel="stylesheet"></head><body><noscript>You need to enable JavaScript to run this app.</noscript><div id="root"></div></body></html>

‎diag/app/build/static/js/main.e117a19e.js‎ renamed to ‎diag/app/build/static/js/main.1d3b50fb.js‎

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎diag/app/build/static/js/main.e117a19e.js.LICENSE.txt‎ renamed to ‎diag/app/build/static/js/main.1d3b50fb.js.LICENSE.txt‎

File renamed without changes.

‎diag/app/build/static/js/main.1d3b50fb.js.map‎

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎diag/app/build/static/js/main.e117a19e.js.map‎

Lines changed: 0 additions & 1 deletion
This file was deleted.

‎diag/app/package-lock.json‎

Lines changed: 3 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎diag/app/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,6 @@
5151
"homepage": ".",
5252
"devDependencies": {
5353
"@types/react-router-bootstrap": "^0.24.5",
54-
"rimraf": "3.0.2"
54+
"rimraf": "^3.0.2"
5555
}
5656
}

‎diag/app/src/Components.tsx‎

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,17 @@ import { WorkflowInstance as Instance } from "./client";
55

66
export function decodePayload(payload: string): string {
77
try {
8-
return atob(payload);
8+
const decoded = atob(payload);
9+
10+
// Try to parse as JSON and pretty-print if valid
11+
try {
12+
JSON.parse(decoded);
13+
// If parsing succeeds, pretty-print the JSON
14+
return JSON.stringify(JSON.parse(decoded), null, 2);
15+
} catch {
16+
// If not valid JSON, return as-is
17+
return decoded;
18+
}
919
} catch {
1020
return payload;
1121
}
@@ -40,6 +50,8 @@ export function decodePayloads(payload: { [key: string]: any }): any {
4050
return r;
4151
}
4252

53+
54+
4355
export const Payload: React.FC<{ payloads: string[] }> = ({ payloads }) => {
4456
return (
4557
<div className="bg-dark text-light rounded p-2">

‎diag/app/src/Instance.tsx‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import {
99
decodePayload,
1010
decodePayloads,
1111
} from "./Components";
12+
import { formatAttributesForDisplay } from "./utils";
1213
import {
1314
ExecutionCompletedAttributes,
1415
ExecutionStartedAttributes,
@@ -208,11 +209,7 @@ function Instance() {
208209
<dd>
209210
<Payload
210211
payloads={[
211-
JSON.stringify(
212-
decodePayloads(event.attributes),
213-
undefined,
214-
2
215-
),
212+
formatAttributesForDisplay(event.attributes)
216213
]}
217214
/>
218215
</dd>

0 commit comments

Comments
 (0)