deprecate credentials file

Author: claytercek

packages/content/docs/contentful-source.md

@@ -9,7 +9,7 @@ Also supports all fields of the Contentful SDK's config.
 See: 'Configuration' under https://contentful.github.io/contentful.js/contentful/9.1.7/
 | Property | Type | Default | Description |
 | - | - | - | - |
-| <a name="module_contentful-source.ContentfulOptions+space">`space`</a> |  <code>string</code>|  <code>''</code>  | Your Contentful space ID. Note that credentials.json will require an accessToken in addition to this |
+| <a name="module_contentful-source.ContentfulOptions+space">`space`</a> |  <code>string</code>|  <code>''</code>  | Your Contentful space ID. Note that an accessToken will be required in addition to this |
 | <a name="module_contentful-source.ContentfulOptions+locale">`locale`</a> |  <code>string</code>|  <code>'en-US'</code>  | Optional. Used to pull localized images. |
 | <a name="module_contentful-source.ContentfulOptions+filename">`filename`</a> |  <code>string</code>|  <code>'content.json'</code>  | Optional. The filename you want to use for where all content (entries and assets metadata) will be stored. |
 | <a name="module_contentful-source.ContentfulOptions+contentTypes">`contentTypes`</a> |  <code>Array.&lt;string&gt;</code>|  | Optionally limit queries to these content types.<br>This will also apply to linked assets.<br>Types that link to other types will include up to 10 levels of child content.<br>E.g. filtering by Story, might also include Chapters and Images.<br>Uses `searchParams['sys.contentType.sys.id[in]']` under the hood. |

packages/content/docs/strapi-source.md

@@ -13,6 +13,6 @@ Options for StrapiSource
 | <a name="module_strapi-source.StrapiOptions+limit">`limit`</a> |  <code>number</code>|  <code>100</code>  | Max number of entries per page. |
 | <a name="module_strapi-source.StrapiOptions+maxNumPages">`maxNumPages`</a> |  <code>number</code>|  <code>-1</code>  | Max number of pages. Use the default of `-1` for all pages |
 | <a name="module_strapi-source.StrapiOptions+pageNumZeroPad">`pageNumZeroPad`</a> |  <code>number</code>|  <code>0</code>  | How many zeros to pad each json filename index with. |
-| <a name="module_strapi-source.StrapiOptions+identifier">`identifier`</a> |  <code>string</code>|  | Username or email. Should be configured via `./credentials.json` |
-| <a name="module_strapi-source.StrapiOptions+password">`password`</a> |  <code>string</code>|  | Should be configured via `./credentials.json` |
+| <a name="module_strapi-source.StrapiOptions+identifier">`identifier`</a> |  <code>string</code>|  | Username or email. Should be configured via `./.env.local` |
+| <a name="module_strapi-source.StrapiOptions+password">`password`</a> |  <code>string</code>|  | Should be configured via `./.env.local` |
 | <a name="module_strapi-source.StrapiOptions+token">`token`</a> |  <code>string</code>|  | Can be used instead of identifer/password if you previously generated one. Otherwise this will be automatically generated using the identifier or password. |

packages/content/lib/content-options.js

@@ -21,7 +21,7 @@ export const TIMESTAMP_TOKEN = '%TIMESTAMP%';
  * @property {Array<Object<string, number>>} [imageTransforms] A list of image transforms to apply to a copy of each downloaded image.
  * @property {Object<string, string>} [contentTransforms] A list of content transforms to apply to all donwloaded content.
  * @property {string} [downloadPath] The path at which to store all downloaded files. Defaults to '.downloads/'.
- * @property {string} [credentialsPath] The path to the json containing credentials for all content sources. Defaults to '.credentials.json'.
+ * @property {string} [credentialsPath] The path to the json containing credentials for all content sources. Defaults to '.credentials.json'. Deprecated in favor of `.env`/`.env.local`.
  * @property {string} [tempPath] Temp file directory path. Defaults to '%DOWNLOAD_PATH%/.tmp/'.
  * @property {string} [backupPath] Temp directory path where all downloaded content will be backed up before removal. Defaults to '%TIMESTAMP%/.tmp-backup/'.
  * @property {string} [keep] Which files to keep in `dest` if `clearOldFilesOnSuccess` or `clearOldFilesOnStart` are `true`. E.g. `'*.json|*.csv|*.xml|*.git*'`
@@ -41,14 +41,13 @@ export const TIMESTAMP_TOKEN = '%TIMESTAMP%';
  */
 
 /**
- * @type {Required<ContentOptions>}
+ * @satisfies {ContentOptions}
  */
 export const CONTENT_OPTION_DEFAULTS = {
 	sources: [],
 	imageTransforms: [],
 	contentTransforms: {},
 	downloadPath: '.downloads/',
-	credentialsPath: '.credentials.json',
 	tempPath: '%DOWNLOAD_PATH%/.tmp/',
 	backupPath: '%TIMESTAMP%/.tmp-backup/',
 	keep: '',

packages/content/lib/content-sources/contentful-source.js

@@ -30,7 +30,7 @@ import { Logger } from '@bluecadet/launchpad-utils';
 
 /**
  * @typedef BaseContentfulOptions
- * @property {string} space Your Contentful space ID. Note that credentials.json will require an accessToken in addition to this
+ * @property {string} space Your Contentful space ID. Note that an accessToken is required in addition to this
  * @property {string} [locale] Optional. Used to pull localized images.
  * @property {string} [filename] Optional. The filename you want to use for where all content (entries and assets metadata) will be stored. Defaults to 'content.json'
  * @property {string} [protocol] Optional. Defaults to 'https'

packages/content/lib/content-sources/strapi-source.js

@@ -26,8 +26,8 @@ import { Logger } from '@bluecadet/launchpad-utils';
 
 /** 
  * @typedef StrapiLoginCredentials
- * @property {string} identifier Username or email. Should be configured via `./credentials.json`
- * @property {string} password Should be configured via `./credentials.json`
+ * @property {string} identifier Username or email. Should be configured via `./.env.local`
+ * @property {string} password Should be configured via `./.env.local`
  *
  * @typedef StrapiTokenCredentials
  * @property {string} token Can be used instead of identifer/password if you previously generated one. Otherwise this will be automatically generated using the identifier or password.

packages/content/lib/credentials.js

@@ -12,20 +12,25 @@ class Credentials {
 	static logger = console;
 
 	/**
-	 * @param {string} credentialsPath 
+	 * @param {string} [credentialsPath] 
 	 * @param {Logger | Console} logger 
 	 */
 	static init(credentialsPath, logger = console) {
 		this.logger = logger;
+
+		if (!credentialsPath) {
+			return;
+		}
+
+		this.logger.warn(`${chalk.white('credentialsPath')} option is deprecated. Please use ${chalk.white('.env')}/${chalk.white('.env.local')} instead.`);
+
 		try {
-			if (credentialsPath && fsx.existsSync(credentialsPath)) {
+			if (fsx.existsSync(credentialsPath)) {
 				this.logger.info(chalk.gray(`Loading credentials from '${chalk.white(credentialsPath)}'`));
 				const rawdata = fsx.readFileSync(credentialsPath);
 				creds = JSON.parse(rawdata.toString());
-			} else if (credentialsPath) {
-				this.logger.warn(`No credentials file found at '${credentialsPath}'`);
 			} else {
-				this.logger.warn(chalk.yellow('No credentials path specified'));
+				this.logger.warn(`No credentials file found at '${credentialsPath}'`);
 			}
 		} catch (err) {
 			if (err instanceof Error) {

packages/content/lib/launchpad-content.js

@@ -60,7 +60,7 @@ export class LaunchpadContent {
 		}
 	}
 
-	/** @type {Required<import('./content-options.js').ResolvedContentOptions>} */
+	/** @type {import('./content-options.js').ResolvedContentOptions} */
 	_config;
 
 	/** @type {Logger} */