chore: remove unnecessary debug logging
Frontend: - Remove 4 console.log statements from terminal.tsx that flooded the browser console with WebSocket traffic (open, received X bytes, sending Y, xterm focused) Backend: - Downgrade Dockerfile/entrypoint compilation logs from INFO to DEBUG in _prepare_manifest_instance - Remove hex-dump diagnostic logging from docker_build.py (was for troubleshooting the backslash continuation bug, now fixed) - Downgrade Dockerfile write log from INFO to DEBUG
This commit is contained in:
@@ -1104,14 +1104,11 @@ async def _prepare_manifest_instance(
|
|||||||
dockerfile = compile_dockerfile(manifest)
|
dockerfile = compile_dockerfile(manifest)
|
||||||
entrypoint = compile_entrypoint(manifest)
|
entrypoint = compile_entrypoint(manifest)
|
||||||
|
|
||||||
logger.info(
|
logger.debug(
|
||||||
"Compiled Dockerfile for instance %s (%d chars)",
|
"Compiled Dockerfile for instance %s (%d chars)",
|
||||||
instance.id,
|
instance.id,
|
||||||
len(dockerfile),
|
len(dockerfile),
|
||||||
)
|
)
|
||||||
logger.debug("Dockerfile content: %s", dockerfile)
|
|
||||||
logger.info("Compiled entrypoint for instance %s", instance.id)
|
|
||||||
logger.debug("Entrypoint content: %s", entrypoint)
|
|
||||||
|
|
||||||
build_ctx = {
|
build_ctx = {
|
||||||
"Dockerfile": dockerfile,
|
"Dockerfile": dockerfile,
|
||||||
|
|||||||
@@ -31,13 +31,7 @@ def build_image(
|
|||||||
# Write Dockerfile
|
# Write Dockerfile
|
||||||
dockerfile_path = Path(instance_dir) / "Dockerfile"
|
dockerfile_path = Path(instance_dir) / "Dockerfile"
|
||||||
dockerfile_path.write_text(dockerfile, newline="\n")
|
dockerfile_path.write_text(dockerfile, newline="\n")
|
||||||
logger.info("Wrote Dockerfile to %s (%d bytes)", dockerfile_path, len(dockerfile))
|
logger.debug("Wrote Dockerfile to %s (%d bytes)", dockerfile_path, len(dockerfile))
|
||||||
logger.debug("Dockerfile content:\n%s", dockerfile)
|
|
||||||
|
|
||||||
# Hex-dump first 8 lines so we can see exactly what bytes Docker receives
|
|
||||||
lines_for_hex = dockerfile.split("\n")[:8]
|
|
||||||
for i, line in enumerate(lines_for_hex, start=1):
|
|
||||||
logger.info("Dockerfile line %d hex: %s", i, line.encode("utf-8").hex())
|
|
||||||
|
|
||||||
# Write build context files
|
# Write build context files
|
||||||
if build_context:
|
if build_context:
|
||||||
|
|||||||
@@ -107,7 +107,6 @@ export const TerminalComponent = React.forwardRef<TerminalRef, TerminalProps>(
|
|||||||
wsRef.current = ws;
|
wsRef.current = ws;
|
||||||
|
|
||||||
ws.onopen = () => {
|
ws.onopen = () => {
|
||||||
console.log(`[Terminal ${sessionId ?? "default"}] WebSocket opened`);
|
|
||||||
setStatus("connected");
|
setStatus("connected");
|
||||||
setError(null);
|
setError(null);
|
||||||
reconnectAttemptsRef.current = 0;
|
reconnectAttemptsRef.current = 0;
|
||||||
@@ -139,10 +138,6 @@ export const TerminalComponent = React.forwardRef<TerminalRef, TerminalProps>(
|
|||||||
if (!termRef.current) return;
|
if (!termRef.current) return;
|
||||||
|
|
||||||
if (event.data instanceof Blob) {
|
if (event.data instanceof Blob) {
|
||||||
// eslint-disable-next-line no-console
|
|
||||||
console.log(
|
|
||||||
`[Terminal ${sessionId ?? "default"}] received ${(event.data as Blob).size} bytes`,
|
|
||||||
);
|
|
||||||
event.data.arrayBuffer().then((buffer) => {
|
event.data.arrayBuffer().then((buffer) => {
|
||||||
const data = new Uint8Array(buffer);
|
const data = new Uint8Array(buffer);
|
||||||
termRef.current?.write(data);
|
termRef.current?.write(data);
|
||||||
@@ -313,9 +308,6 @@ export const TerminalComponent = React.forwardRef<TerminalRef, TerminalProps>(
|
|||||||
// Open xterm first (must happen before fit)
|
// Open xterm first (must happen before fit)
|
||||||
term.open(container);
|
term.open(container);
|
||||||
term.focus();
|
term.focus();
|
||||||
console.log(
|
|
||||||
`[Terminal ${sessionId ?? "default"}] xterm opened and focused`,
|
|
||||||
);
|
|
||||||
const ws = connectWebSocket();
|
const ws = connectWebSocket();
|
||||||
|
|
||||||
// Initial fit after layout settles (terminal must be opened first)
|
// Initial fit after layout settles (terminal must be opened first)
|
||||||
@@ -340,11 +332,6 @@ export const TerminalComponent = React.forwardRef<TerminalRef, TerminalProps>(
|
|||||||
|
|
||||||
// Handle terminal input
|
// Handle terminal input
|
||||||
term.onData((data) => {
|
term.onData((data) => {
|
||||||
// eslint-disable-next-line no-console
|
|
||||||
console.log(
|
|
||||||
`[Terminal ${sessionId ?? "default"}] sending:`,
|
|
||||||
JSON.stringify(data),
|
|
||||||
);
|
|
||||||
const currentWs = wsRef.current;
|
const currentWs = wsRef.current;
|
||||||
if (currentWs?.readyState !== WebSocket.OPEN) return;
|
if (currentWs?.readyState !== WebSocket.OPEN) return;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user