Skip to content

Commit d823470

Browse files
committed
refactor(cli): move loglevel to log.level
1 parent 24dcdf7 commit d823470

4 files changed

Lines changed: 12 additions & 7 deletions

File tree

packages/cli/src/cli.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ program.action(async function (_, options, cmd) {
131131

132132
try {
133133
const config = await buildConfigFromFile(options, defaultFilePath, env, schemaValidator);
134-
setLogLevel(options.logLevel ?? config.logLevel);
134+
setLogLevel(options.logLevel ?? config.log.level);
135135
config.servient.clientOnly = options.clientOnly ?? config.servient.clientOnly;
136136
servient = new DefaultServient(config);
137137
} catch (err) {

packages/cli/src/configuration.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ export const defaultConfiguration = Object.freeze({
5454
port: 5683,
5555
},
5656
credentials: {},
57-
logLevel: "warn",
57+
log: { level: "warn" },
5858
} as const satisfies Configuration);
5959

6060
export type ConfigurationAfterDefaults = Merge<Configuration, Generalize<Mutable<typeof defaultConfiguration>>>;

packages/cli/src/wot-servient-schema.conf.json

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -124,9 +124,14 @@
124124
},
125125
"additionalProperties": false
126126
},
127-
"logLevel": {
128-
"type": "string",
129-
"enum": ["debug", "info", "warn", "error"]
127+
"log": {
128+
"type": "object",
129+
"properties": {
130+
"level": {
131+
"type": "string",
132+
"enum": ["debug", "info", "warn", "error"]
133+
}
134+
}
130135
}
131136
},
132137
"additionalProperties": false

packages/cli/test/configuration.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ class ConfigurationTest {
5454
expect(result).to.have.property("http");
5555
expect(result.http.port).to.equal(8080);
5656
expect(result.coap.port).to.equal(5683);
57-
expect(result.logLevel).to.equal("warn");
57+
expect(result.log.level).to.equal("warn");
5858
}
5959

6060
@test async "should handle credentials in config"() {
@@ -101,7 +101,7 @@ class ConfigurationTest {
101101

102102
expect(result.http.port).to.equal(8888);
103103
expect(result.coap.port).to.equal(defaultConfiguration.coap.port);
104-
expect(result.logLevel).to.equal(defaultConfiguration.logLevel);
104+
expect(result.log.level).to.equal(defaultConfiguration.log.level);
105105
}
106106

107107
@test async "should read and build config from file"() {

0 commit comments

Comments
 (0)