Chore: Upgrade to AIK & Massive Overhaul to internal processing #78
|
|
@ -45,10 +45,7 @@
|
|||
"exports": {
|
||||
".": "./src/index.ts",
|
||||
"./api": "./src/api/index.ts",
|
||||
"./404.astro": "./src/routes/404.astro",
|
||||
"./rss-routes/*": "./src/integrations/rssfeed/routes/*",
|
||||
"./satoriOG": "./src/integrations/satoriog/satori.ts",
|
||||
"./open-graph/*": "./src/integrations/satoriog/routes/*"
|
||||
"./satoriOG": "./src/integrations/satoriog/satori.ts"
|
||||
},
|
||||
"scripts": {
|
||||
"test": "vitest run",
|
||||
|
|
|
|||
|
|
@ -96,7 +96,7 @@ export default defineIntegration({
|
|||
|
|
||||
routeLogInfo(c.bold(c.cyan(`Setting up ${routename} route`)));
|
||||
injectRoute({
|
||||
pattern: pattern,
|
||||
entrypoint: `${name}${entrypoint}`,
|
||||
```suggestion
} else if (verbose) {
GhostIntegrationLogger.info(c.gray("Theme Provider is disabled"));
}
```
same here same here
same here same here
same here same here
is there a reason for not importing it at the top of the file? if yes, I think you can use is there a reason for not importing it at the top of the file? if yes, I think you can use `resolve` from `createResolver` for the path
This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D
ah this pulls the version from the user's project? ah this pulls the version from the user's project?
I ALWAYS FORGET YOU CAN DO I ALWAYS FORGET YOU CAN DO `else if` vscode should complain if you do an `else { if()` lol
i think if you have eslint you can enable such a rule i think if you have eslint you can enable such a rule
grabs the current installed one from grabs the current installed one from `astro-ghostcms` local version and compares to the latestversion from NPM :P I did actually try to use the `resolve()` util but that just grabbed the playgrounds package.json ironically...
Instead of all these manual checks for Or could be a Florian special and return a console log function: Instead of all these manual checks for `verbose` you'd probably be better off with a util function which checks it instead.
```ts
const log = message => {
if (verbose) {
console.log(...)
}
}
```
Or could be a Florian special and return a console log function:
```ts
const createLogger = verbose => message => {
if (verbose) {
console.log(...)
}
}
const log = createLogger(options.verbose)
log(whatever) // now you don't have to check everywhere
```
added in new commit, since im not using console.log had to modify it for my usage a little added in new commit, since im not using console.log had to modify it for my usage a little
Okay i fixed that issue. Now using Okay i fixed that issue. Now using `resolve()` properly 😄
This logging function could also be extracted This logging function could also be extracted
```ts
const formattedMessage = message => `${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`
```
Updated... Updated...
```ts
// Configure Loggers
const GhostLogger = logger.fork(c.bold(c.blue("👻 Astro-GhostCMS")));
const loggerTagged = (message: string) => {
return logger.fork(`${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`)
}
// Configure ENV Logger
const GhostENVLogger = loggerTagged("ENV Check");
// Configure Integration Loggers & verbose logging
const GhostIntegrationLogger = loggerTagged("Integrations");
// Configure Route Logger & verbose logging
const GhostRouteLogger = loggerTagged("Router");
```
|
||||
entrypoint: resolve(`./routes${entrypoint}`),
|
||||
```suggestion
} else if (verbose) {
GhostIntegrationLogger.info(c.gray("Theme Provider is disabled"));
}
```
same here same here
same here same here
same here same here
is there a reason for not importing it at the top of the file? if yes, I think you can use is there a reason for not importing it at the top of the file? if yes, I think you can use `resolve` from `createResolver` for the path
This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D
ah this pulls the version from the user's project? ah this pulls the version from the user's project?
I ALWAYS FORGET YOU CAN DO I ALWAYS FORGET YOU CAN DO `else if` vscode should complain if you do an `else { if()` lol
i think if you have eslint you can enable such a rule i think if you have eslint you can enable such a rule
grabs the current installed one from grabs the current installed one from `astro-ghostcms` local version and compares to the latestversion from NPM :P I did actually try to use the `resolve()` util but that just grabbed the playgrounds package.json ironically...
Instead of all these manual checks for Or could be a Florian special and return a console log function: Instead of all these manual checks for `verbose` you'd probably be better off with a util function which checks it instead.
```ts
const log = message => {
if (verbose) {
console.log(...)
}
}
```
Or could be a Florian special and return a console log function:
```ts
const createLogger = verbose => message => {
if (verbose) {
console.log(...)
}
}
const log = createLogger(options.verbose)
log(whatever) // now you don't have to check everywhere
```
added in new commit, since im not using console.log had to modify it for my usage a little added in new commit, since im not using console.log had to modify it for my usage a little
Okay i fixed that issue. Now using Okay i fixed that issue. Now using `resolve()` properly 😄
This logging function could also be extracted This logging function could also be extracted
```ts
const formattedMessage = message => `${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`
```
Updated... Updated...
```ts
// Configure Loggers
const GhostLogger = logger.fork(c.bold(c.blue("👻 Astro-GhostCMS")));
const loggerTagged = (message: string) => {
return logger.fork(`${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`)
}
// Configure ENV Logger
const GhostENVLogger = loggerTagged("ENV Check");
// Configure Integration Loggers & verbose logging
const GhostIntegrationLogger = loggerTagged("Integrations");
// Configure Route Logger & verbose logging
const GhostRouteLogger = loggerTagged("Router");
```
|
||||
prerender: true,
|
||||
});
|
||||
} else {
|
||||
|
|
|
|||
```suggestion
} else if (verbose) {
GhostIntegrationLogger.info(c.gray("Theme Provider is disabled"));
}
```
```suggestion
} else if (verbose) {
GhostIntegrationLogger.info(c.gray("Theme Provider is disabled"));
}
```
same here same here
same here same here
same here same here
same here same here
same here same here
same here same here
is there a reason for not importing it at the top of the file? if yes, I think you can use is there a reason for not importing it at the top of the file? if yes, I think you can use `resolve` from `createResolver` for the path
is there a reason for not importing it at the top of the file? if yes, I think you can use is there a reason for not importing it at the top of the file? if yes, I think you can use `resolve` from `createResolver` for the path
This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D
This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D
ah this pulls the version from the user's project? ah this pulls the version from the user's project?
ah this pulls the version from the user's project? ah this pulls the version from the user's project?
I ALWAYS FORGET YOU CAN DO I ALWAYS FORGET YOU CAN DO `else if` vscode should complain if you do an `else { if()` lol
I ALWAYS FORGET YOU CAN DO I ALWAYS FORGET YOU CAN DO `else if` vscode should complain if you do an `else { if()` lol
i think if you have eslint you can enable such a rule i think if you have eslint you can enable such a rule
i think if you have eslint you can enable such a rule i think if you have eslint you can enable such a rule
grabs the current installed one from grabs the current installed one from `astro-ghostcms` local version and compares to the latestversion from NPM :P I did actually try to use the `resolve()` util but that just grabbed the playgrounds package.json ironically...
grabs the current installed one from grabs the current installed one from `astro-ghostcms` local version and compares to the latestversion from NPM :P I did actually try to use the `resolve()` util but that just grabbed the playgrounds package.json ironically...
Instead of all these manual checks for Or could be a Florian special and return a console log function: Instead of all these manual checks for `verbose` you'd probably be better off with a util function which checks it instead.
```ts
const log = message => {
if (verbose) {
console.log(...)
}
}
```
Or could be a Florian special and return a console log function:
```ts
const createLogger = verbose => message => {
if (verbose) {
console.log(...)
}
}
const log = createLogger(options.verbose)
log(whatever) // now you don't have to check everywhere
```
Instead of all these manual checks for Or could be a Florian special and return a console log function: Instead of all these manual checks for `verbose` you'd probably be better off with a util function which checks it instead.
```ts
const log = message => {
if (verbose) {
console.log(...)
}
}
```
Or could be a Florian special and return a console log function:
```ts
const createLogger = verbose => message => {
if (verbose) {
console.log(...)
}
}
const log = createLogger(options.verbose)
log(whatever) // now you don't have to check everywhere
```
added in new commit, since im not using console.log had to modify it for my usage a little added in new commit, since im not using console.log had to modify it for my usage a little
added in new commit, since im not using console.log had to modify it for my usage a little added in new commit, since im not using console.log had to modify it for my usage a little
Okay i fixed that issue. Now using Okay i fixed that issue. Now using `resolve()` properly 😄
Okay i fixed that issue. Now using Okay i fixed that issue. Now using `resolve()` properly 😄
This logging function could also be extracted This logging function could also be extracted
```ts
const formattedMessage = message => `${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`
```
This logging function could also be extracted This logging function could also be extracted
```ts
const formattedMessage = message => `${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`
```
Updated... Updated...
```ts
// Configure Loggers
const GhostLogger = logger.fork(c.bold(c.blue("👻 Astro-GhostCMS")));
const loggerTagged = (message: string) => {
return logger.fork(`${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`)
}
// Configure ENV Logger
const GhostENVLogger = loggerTagged("ENV Check");
// Configure Integration Loggers & verbose logging
const GhostIntegrationLogger = loggerTagged("Integrations");
// Configure Route Logger & verbose logging
const GhostRouteLogger = loggerTagged("Router");
```
Updated... Updated...
```ts
// Configure Loggers
const GhostLogger = logger.fork(c.bold(c.blue("👻 Astro-GhostCMS")));
const loggerTagged = (message: string) => {
return logger.fork(`${c.bold(c.blue("👻 Astro-GhostCMS"))}${c.gray("/")}${c.blue(message)}`)
}
// Configure ENV Logger
const GhostENVLogger = loggerTagged("ENV Check");
// Configure Integration Loggers & verbose logging
const GhostIntegrationLogger = loggerTagged("Integrations");
// Configure Route Logger & verbose logging
const GhostRouteLogger = loggerTagged("Router");
```
|
||||
|
|
@ -23,16 +23,14 @@ export default defineIntegration({
|
|||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
|
||||
RSSLogger.info(c.bold(c.magenta("RSS Feed Enabled. Setting up...")));
|
||||
|
||||
const rssRoute = "@matthiesenxyz/astro-ghostcms/rss-routes";
|
||||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
|
||||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
injectRoute({
|
||||
pattern: "/rss-style.xsl",
|
||||
entrypoint: `${rssRoute}/rss-style.xsl.ts`,
|
||||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
entrypoint: resolve("./routes/rss-style.xsl.ts"),
|
||||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
});
|
||||
|
||||
injectRoute({
|
||||
pattern: "/rss.xml",
|
||||
entrypoint: `${rssRoute}/rss.xml.ts`,
|
||||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
entrypoint: resolve("./routes/rss.xml.ts"),
|
||||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
});
|
||||
},
|
||||
"astro:config:done": ({ logger }) => {
|
||||
|
|
|
|||
|
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
You can probably resolve these paths instead of exporting it from your package? You can probably resolve these paths instead of exporting it from your package?
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
I was told that any Injected route had to be from an export.... I'd be interested in that alternative... I was told that any Injected route had to be from an export.... I'd be interested in that alternative...
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
They did in the old days before They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
Yeah.... Thats updated now across the board... and those exports for Yeah.... Thats updated now across the board... and those exports for `/rss-routes` and `/open-graph` are now moved to `resolve()`
|
||||
|
|
@ -31,36 +31,34 @@ export default defineIntegration({
|
|||
c.bold(c.magenta("OG Image Integration Enabled. Setting up...")),
|
||||
);
|
||||
|
||||
const pkgname = "@matthiesenxyz/astro-ghostcms/open-graph";
|
||||
|
||||
injectRoute({
|
||||
pattern: "/open-graph/[slug].png",
|
||||
entrypoint: `${pkgname}/[slug].png.ts`,
|
||||
entrypoint: resolve("./routes/[slug].png.ts"),
|
||||
});
|
||||
|
||||
injectRoute({
|
||||
pattern: "/open-graph/index.png",
|
||||
entrypoint: `${pkgname}/index.png.ts`,
|
||||
entrypoint: resolve("./routes/index.png.ts"),
|
||||
});
|
||||
|
||||
injectRoute({
|
||||
pattern: "/open-graph/authors.png",
|
||||
entrypoint: `${pkgname}/authors.png.ts`,
|
||||
entrypoint: resolve("./routes/authors.png.ts"),
|
||||
});
|
||||
|
||||
injectRoute({
|
||||
pattern: "/open-graph/author/[slug].png",
|
||||
entrypoint: `${pkgname}/author/[slug].png.ts`,
|
||||
entrypoint: resolve("./routes/author/[slug].png.ts"),
|
||||
});
|
||||
|
||||
injectRoute({
|
||||
pattern: "/open-graph/tags.png",
|
||||
entrypoint: `${pkgname}/tags.png.ts`,
|
||||
entrypoint: resolve("./routes/tags.png.ts"),
|
||||
});
|
||||
|
||||
injectRoute({
|
||||
pattern: "/open-graph/tag/[slug].png",
|
||||
entrypoint: `${pkgname}/tag/[slug].png.ts`,
|
||||
entrypoint: resolve("./routes/tag/[slug].png.ts"),
|
||||
});
|
||||
|
||||
updateConfig({
|
||||
|
|
|
|||
|
|
@ -14,7 +14,7 @@ export default defineConfig({
|
|||
ThemeProvider: {
|
||||
theme: "@matthiesenxyz/astro-ghostcms-brutalbyelian",
|
||||
},
|
||||
verbose: true,
|
||||
verbose: false,
|
||||
}),
|
||||
],
|
||||
});
|
||||
|
|
|
|||
same here
same here
same here
same here
same here
same here
is there a reason for not importing it at the top of the file? if yes, I think you can use
resolvefromcreateResolverfor the pathis there a reason for not importing it at the top of the file? if yes, I think you can use
resolvefromcreateResolverfor the pathThis is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D
This is simply to display any new version during dev-mode only... There is a check for latest version and it compares the local version :D
ah this pulls the version from the user's project?
ah this pulls the version from the user's project?
I ALWAYS FORGET YOU CAN DO
else ifvscode should complain if you do anelse { if()lolI ALWAYS FORGET YOU CAN DO
else ifvscode should complain if you do anelse { if()loli think if you have eslint you can enable such a rule
i think if you have eslint you can enable such a rule
grabs the current installed one from
astro-ghostcmslocal version and compares to the latestversion from NPM :P I did actually try to use theresolve()util but that just grabbed the playgrounds package.json ironically...grabs the current installed one from
astro-ghostcmslocal version and compares to the latestversion from NPM :P I did actually try to use theresolve()util but that just grabbed the playgrounds package.json ironically...Instead of all these manual checks for
verboseyou'd probably be better off with a util function which checks it instead.Or could be a Florian special and return a console log function:
Instead of all these manual checks for
verboseyou'd probably be better off with a util function which checks it instead.Or could be a Florian special and return a console log function:
added in new commit, since im not using console.log had to modify it for my usage a little
added in new commit, since im not using console.log had to modify it for my usage a little
Okay i fixed that issue. Now using
resolve()properly 😄Okay i fixed that issue. Now using
resolve()properly 😄This logging function could also be extracted
This logging function could also be extracted
Updated...
Updated...