Skip to content
This repository was archived by the owner on Jul 30, 2025. It is now read-only.

Commit 364456c

Browse files
committed
fix: remove ancient pretty printers for ancient usage model
BREAKING CHANGE: this removes some of the exported pretty printers from kui-shell/core
1 parent 5c0fe58 commit 364456c

File tree

4 files changed

+5
-145
lines changed

4 files changed

+5
-145
lines changed

packages/core/src/core/usage/pretty-code.ts

Lines changed: 0 additions & 95 deletions
This file was deleted.

packages/core/src/core/usage/render-usage.ts

Lines changed: 0 additions & 35 deletions
This file was deleted.

packages/core/src/index.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,7 @@ export { isUsageError, UsageError, UsageModel, UsageRow } from './core/usage-err
5959
export { isMessageWithUsageModel, isMessageWithCode } from './core/usage-error'
6060

6161
// ascii usage
62-
export { default as renderUsage } from './core/usage/render-usage'
6362
export { default as PrettyUsageModel } from './core/usage/types'
64-
import * as pc from './core/usage/pretty-code'
65-
export { pc }
6663

6764
// eventChannelUnsafe
6865
export * from './api'

plugins/plugin-s3/src/browse/index.ts

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -14,23 +14,16 @@
1414
* limitations under the License.
1515
*/
1616

17-
import { Arguments, PrettyUsageModel, Registrar, encodeComponent, renderUsage } from '@kui-shell/core'
18-
19-
import * as Usage from './usage'
17+
import { Arguments, Registrar, encodeComponent } from '@kui-shell/core'
2018

2119
const opts = { needsUI: true, width: 720, height: 900 }
2220

23-
function doBrowse(this: string, usage: PrettyUsageModel, { REPL, parsedOptions }: Arguments) {
24-
if (parsedOptions.h || parsedOptions.help) {
25-
return renderUsage(usage)
26-
}
27-
21+
function doBrowse(this: string, { REPL }: Arguments) {
2822
return REPL.qexec(`ls ${encodeComponent(this)}`)
2923
}
3024

3125
export default async function(registrar: Registrar) {
32-
registrar.listen('/browse', renderUsage.bind(undefined, Usage.browse))
33-
registrar.listen('/browse/s3', doBrowse.bind('/s3', Usage.s3), opts)
34-
registrar.listen('/browse/cc', doBrowse.bind('/s3/aws/commoncrawl', Usage.cc('cc')), opts)
35-
registrar.listen('/browse/commoncrawl', doBrowse.bind('/s3/aws/commoncrawl', Usage.cc('commoncrawl')), opts)
26+
registrar.listen('/browse/s3', doBrowse.bind('/s3'), opts)
27+
registrar.listen('/browse/cc', doBrowse.bind('/s3/aws/commoncrawl'), opts)
28+
registrar.listen('/browse/commoncrawl', doBrowse.bind('/s3/aws/commoncrawl'), opts)
3629
}

0 commit comments

Comments
 (0)