Skip to content

revert: roll back output truncation/key-value heuristic from #1081#1086

Closed
jackwener wants to merge 1 commit intomainfrom
revert-1081-output-policy
Closed

revert: roll back output truncation/key-value heuristic from #1081#1086
jackwener wants to merge 1 commit intomainfrom
revert-1081-output-policy

Conversation

@jackwener
Copy link
Copy Markdown
Owner

Summary

Revert 3bbea01 from PR #1081.

Why

Follow-up discussion after merge concluded that the change took the wrong first-principles direction:

  • it treated output design as a table-renderer truncation problem
  • it hardcoded single-row => key/value as a global renderer heuristic
  • the desired direction is to define presentation by result semantics / command shape first, then render

So this PR removes the truncation + single-row heuristic patch from src/output.ts and restores the prior behavior.

Verification

  • node node_modules/vitest/vitest.mjs run src/output.test.ts
  • npm run typecheck

@jackwener
Copy link
Copy Markdown
Owner Author

Closing as duplicate of #1085 (merged first, CI CLEAN). Thanks @codex-mini1 for the fast reaction — #1085 was opened 29s earlier by @First-principles-1 per task #59. Only one revert needed.

@jackwener jackwener closed this Apr 19, 2026
@jackwener jackwener deleted the revert-1081-output-policy branch April 19, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant