dev: fix recent notes framework

Author: virtualguard101

src/mkdocs_note/plugin.py

@@ -2,7 +2,6 @@
 
 import os
 import json
-from pathlib import Path
 
 from mkdocs.structure.files import Files, File
 from mkdocs.config.defaults import MkDocsConfig
@@ -150,20 +149,15 @@ def on_page_markdown(
 		Returns:
 			str: The markdown content.
 		"""
-		if self.config.recent_notes_config["enabled"]:
-			if self.is_note_index_page(page.file):
-				markdown = insert_recent_note_links(
-					markdown=markdown,
-					notes_list=self.notes_list,
-					insert_num=self.config.recent_notes_config["insert_num"],
-					replace_marker=self.config.recent_notes_config["insert_marker"],
-				)
-			else:
-				log.warning("Recent notes are not supported on non-note index pages.")
-				markdown = markdown
-		else:
-			log.debug("Recent notes insertion are disabled.")
-			markdown = markdown
+		# Only process recent notes on the note index page
+		if self.config.recent_notes_config["enabled"] and self.is_note_index_page(page.file):
+			markdown = insert_recent_note_links(
+				markdown=markdown,
+				notes_list=self.notes_list,
+				insert_num=self.config.recent_notes_config["insert_num"],
+				replace_marker=self.config.recent_notes_config["insert_marker"],
+			)
+			log.debug(f"Recent notes inserted into {page.file.src_uri}")
 
 		return markdown
 
@@ -172,12 +166,14 @@ def is_note_index_page(self, f: File) -> bool:
 		"""Check if the page is a note index page.
 
 		Args:
-			page (Page): The page to check.
+			f (File): The file to check.
 
 		Returns:
 			bool: True if the page is a note index page, False otherwise.
 		"""
-		return f.src_uri == str(Path(self.config.notes_root) / "index.md")
+		# src_uri is relative to docs_dir, so we just check for 'index.md'
+		# when notes_root is the docs_dir itself
+		return f.src_uri == "index.md"
 
 
 def insert_recent_note_links(
@@ -198,9 +194,13 @@ def insert_recent_note_links(
         str: The markdown content with recent note links inserted.
     """
 
-    content = ""
+    content = "<ul>\n"
     for f in notes_list[:insert_num]:
         title = extract_title(f)
         date = extract_date(f).strftime("%Y-%m-%d %H:%M:%S")
-        content += f"- <div class='recent-notes'><a href='{f.page.abs_url}'>{title}</a><small>{date}</small></div>\n"
+        # Use f.url (relative URL) or f.page.url if page is available
+        url = f.page.url if hasattr(f, 'page') and f.page else f.url
+        # No indentation to avoid Markdown treating it as code block
+        content += f'<li><div style="display:flex; justify-content:space-between; align-items:center;"><a href="{url}">{title}</a><span style="font-size:0.8em; color:#888;">{date}</span></div></li>\n'
+    content += "</ul>\n"
     return markdown.replace(replace_marker, content)

src/mkdocs_note/utils/scanner.py

@@ -20,23 +20,33 @@ def scan_notes(files: Files, config) -> tuple[list[File], list[File]]:
 	"""
 	notes_dir = Path(config.notes_root) if isinstance(config.notes_root, str) else config.notes_root
 	if not notes_dir.exists():
+		logger.warning(f"Notes directory does not exist: {notes_dir}")
 		return [], []
 
 	notes = []
 	invalid_files = []
 
 	try:
 		for f in files:
-			path_name = f.src_uri.split("/")
-
-			if len(path_name) < 2 or path_name[1] != notes_dir:
+			# Skip non-documentation pages
+			if not f.is_documentation_page():
+				continue
+			
+			# Check if file is within notes_root by comparing absolute paths
+			# f.abs_src_path is the absolute path to the source file
+			try:
+				file_path = Path(f.abs_src_path)
+				# Check if the file is within the notes_root directory
+				file_path.relative_to(notes_dir)
+			except (ValueError, AttributeError):
+				# File is not within notes_root
 				continue
 
-			if f.is_documentation_page() and validate_frontmatter(f):
+			# Validate frontmatter
+			if validate_frontmatter(f):
 				notes.append(f)
 			else:
 				invalid_files.append(f)
-				continue
 	except Exception as e:
 		logger.error(f"Error scanning notes: {e}")
 		raise e