Merge pull request #916 from gewarren/refresh

Fix metadata refresh functionality

Author: bharney0

packages/docs-markdown/src/controllers/metadata/metadata-controller.ts

@@ -39,7 +39,7 @@ export function getAllEffectiveMetadata(docFxFileInfo: DocFxFileInfo): MetadataE
 		const result = results.find(r => !!r.groups && r.groups.metadata);
 		if (result) {
 			try {
-				const metadataJson = jsyaml.load(result.groups.metadata);
+				const metadataJson = jsyaml.load(result.groups.metadata, { skipInvalid: true });
 				if (metadataJson) {
 					const lines = result.groups.metadata.split(/\r\n|\n\r|\n|\r/);
 					for (const [key, value] of Object.entries(metadataJson)) {

packages/docs-markdown/src/extension.ts

@@ -201,7 +201,9 @@ export async function activate(context: ExtensionContext) {
 			}
 		}
 	});
-	workspace.onDidChangeTextDocument(e => {
+
+	// Refresh when the active file is changed, including closing it.
+	window.onDidChangeActiveTextEditor(e => {
 		metadataTreeProvider.refresh();
 	});