Chore: Upgrade to AIK & Massive Overhaul to internal processing #78

Merged
Adammatthiesen merged 46 commits from issue-77-Chore_Upgrade_to_utilize_AIK_&_Massive_integration_overhaul into main 2024-03-07 12:08:19 +00:00
3 changed files with 30 additions and 21 deletions
Showing only changes of commit 41b0ce75d3 - Show all commits

View File

@ -41,10 +41,13 @@ export default defineIntegration({
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
"RSSGenerator",
)}`,
);
const verboseLogsInfo = (message:string) => {
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
if (options.verbose) {
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
RSSLogger.info(message);
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
}
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
};
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
if (options.verbose) {
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
RSSLogger.info(c.bold(c.green("RSS Feed Setup Complete")));
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
}
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
verboseLogsInfo(c.bold(c.green("RSS Feed Setup Complete")));
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
},
};
},

jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
jdtjenkins commented 2024-03-05 08:52:19 +00:00 (Migrated from github.com)
Review

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?
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
Adammatthiesen commented 2024-03-05 09:00:35 +00:00 (Migrated from github.com)
Review

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...
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
jdtjenkins commented 2024-03-05 09:04:42 +00:00 (Migrated from github.com)
Review

They did in the old days before createResolver ❤️ it just can't be a virtual module

They did in the old days before `createResolver` ❤️ it just can't be a virtual module
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`
Adammatthiesen commented 2024-03-05 09:30:57 +00:00 (Migrated from github.com)
Review

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 `/rss-routes` and `/open-graph` are now moved to `resolve()`

View File

@ -73,12 +73,13 @@ export default defineIntegration({
"SatoriOG",
)}`,
);
const verboseLogsInfo = (message:string) => {
if (options.verbose) {
SatoriLogger.info(message);
}
};
if (options.verbose) {
SatoriLogger.info(
c.bold(c.green("OG Image Integration Setup Complete")),
);
}
verboseLogsInfo(c.bold(c.green("OG Image Integration Setup Complete")));
},
};
},

View File

@ -25,24 +25,26 @@ export default defineIntegration({
"Theme Provider",
)}`,
);
const verboseLogsInfo = (message:string) => {
if (options.verbose) {
themeLogger.info(message);
}
};
themeLogger.info(
c.bold(c.magenta("Theme Provider enabled. Setting up...")),
);
if (options.verbose) {
if (options.theme === "@matthiesenxyz/astro-ghostcms-theme-default") {
themeLogger.info(
if (options.theme === "@matthiesenxyz/astro-ghostcms-theme-default") {
verboseLogsInfo(
c.blue("No theme is set, injecting default theme"),
);
} else {
themeLogger.info(
`${c.bold(c.cyan("Injecting Theme:"))} ${c.bold(
c.underline(c.magenta(options.theme)),
)}`,
);
}
} else {
verboseLogsInfo(`${c.bold(c.cyan("Injecting Theme:"))} ${c.bold(c.underline(c.magenta(options.theme)))}`);
}
injectRoute({
pattern: "/",
@ -85,10 +87,13 @@ export default defineIntegration({
"Theme Provider",
)}`,
);
const verboseLogsInfo = (message:string) => {
if (options.verbose) {
themeLogger.info(message);
}
};
if (options.verbose) {
themeLogger.info(c.bold(c.green("Provider Setup Complete")));
}
verboseLogsInfo(c.bold(c.green("Provider Setup Complete")));
},
};
},