fix: resolve config refresh failure with spring.config.import on 2025.0.x

Fix two defects in NacosPropertySourceRefreshListener that prevented
dynamic config refresh when using spring.config.import=nacos:

1. PropertySource name mismatch: ConfigData path uses 'group@dataId'
   naming while RefreshListener looked for 'dataId,group'. Now tries
   both naming conventions.

2. File extension hardcoded: RefreshListener always used 'properties'
   extension, breaking yml/json/xml configs. Now reads actual suffix
   from NacosPropertySource.

Changes:
- Add suffix field to NacosPropertySource to preserve file extension
- Update NacosPropertySourceBuilder and NacosConfigDataLoader to pass suffix
- Update NacosPropertySourceRefreshListener to handle both naming formats
  and use actual file extension

Fixes #4331

Signed-off-by: wushiyuan <wushiyuanwork@outlook.com>

Author: wushiyuanmaimob

spring-cloud-alibaba-starters/spring-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/client/NacosPropertySource.java

@@ -55,19 +55,31 @@ public class NacosPropertySource extends MapPropertySource {
 	 */
 	private final boolean isRefreshable;
 
+	/**
+	 * File extension (suffix) of the config data (e.g., "properties", "yml", "json").
+	 */
+	private final String suffix;
+
 	NacosPropertySource(String group, String dataId, Map<String, Object> source,
-			Date timestamp, boolean isRefreshable) {
+			Date timestamp, boolean isRefreshable, String suffix) {
 		super(String.join(NacosConfigProperties.COMMAS, dataId, group), source);
 		this.group = group;
 		this.dataId = dataId;
 		this.timestamp = timestamp;
 		this.isRefreshable = isRefreshable;
+		this.suffix = suffix;
 	}
 
 	public NacosPropertySource(List<PropertySource<?>> propertySources, String group,
 			String dataId, Date timestamp, boolean isRefreshable) {
 		this(group, dataId, getSourceMap(group, dataId, propertySources), timestamp,
-				isRefreshable);
+				isRefreshable, "properties");
+	}
+
+	public NacosPropertySource(List<PropertySource<?>> propertySources, String group,
+			String dataId, Date timestamp, boolean isRefreshable, String suffix) {
+		this(group, dataId, getSourceMap(group, dataId, propertySources), timestamp,
+				isRefreshable, suffix);
 	}
 
 	private static Map<String, Object> getSourceMap(String group, String dataId,
@@ -129,4 +141,8 @@ public boolean isRefreshable() {
 		return isRefreshable;
 	}
 
+	public String getSuffix() {
+		return suffix;
+	}
+
 }

spring-cloud-alibaba-starters/spring-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/client/NacosPropertySourceBuilder.java

@@ -74,7 +74,7 @@ public NacosPropertySource build(String dataId, String group, String fileExtensi
 		List<PropertySource<?>> propertySources = loadNacosData(dataId, group,
 				fileExtension);
 		NacosPropertySource nacosPropertySource = new NacosPropertySource(propertySources,
-				group, dataId, new Date(), isRefreshable);
+				group, dataId, new Date(), isRefreshable, fileExtension);
 		NacosPropertySourceRepository.collectNacosPropertySource(nacosPropertySource);
 		return nacosPropertySource;
 	}

spring-cloud-alibaba-starters/spring-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/configdata/NacosConfigDataLoader.java

@@ -84,7 +84,7 @@ public ConfigData doLoad(ConfigDataLoaderContext context,
 
 			NacosPropertySource propertySource = new NacosPropertySource(propertySources,
 					config.getGroup(), config.getDataId(), new Date(),
-					config.isRefreshEnabled());
+					config.isRefreshEnabled(), config.getSuffix());
 
 			NacosPropertySourceRepository.collectNacosPropertySource(propertySource);
 

spring-cloud-alibaba-starters/spring-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/refresh/NacosPropertySourceRefreshListener.java

@@ -100,12 +100,32 @@ public void handle(NacosConfigRefreshEvent event) {
 
 				NacosPropertySourceBuilder nacosPropertySourceBuilder = new NacosPropertySourceBuilder(nacosConfigManager.getConfigService(), nacosConfigManager.getNacosConfigProperties()
 						.getTimeout());
-				String sourceName = String.join(NacosConfigProperties.COMMAS, event.dataId, event.group);
+
+				// Try both naming conventions: bootstrap path (dataId,group) and ConfigData path (group@dataId)
+				String bootstrapName = String.join(NacosConfigProperties.COMMAS, event.dataId, event.group);
+				String configDataName = event.group + "@" + event.dataId;
+
 				ConfigurableEnvironment environment = ((ConfigurableApplicationContext) applicationContext).getEnvironment();
 				MutablePropertySources target = environment.getPropertySources();
-				PropertySource<?> prevpropertySource = target.get(sourceName);
+
+				PropertySource<?> prevpropertySource = target.get(bootstrapName);
+				String sourceName = bootstrapName;
+
+				// If not found with bootstrap naming, try ConfigData naming
+				if (prevpropertySource == null) {
+					prevpropertySource = target.get(configDataName);
+					sourceName = configDataName;
+				}
+
 				if (prevpropertySource instanceof NacosPropertySource) {
-					NacosPropertySource newProperSource = nacosPropertySourceBuilder.build(event.getDataId(), event.getGroup(), "properties", ((NacosPropertySource) prevpropertySource).isRefreshable());
+					NacosPropertySource prevNacosSource = (NacosPropertySource) prevpropertySource;
+					// Use the actual suffix from the previous source instead of hardcoding "properties"
+					String fileExtension = prevNacosSource.getSuffix();
+					if (fileExtension == null || fileExtension.isEmpty()) {
+						fileExtension = "properties";
+					}
+					NacosPropertySource newProperSource = nacosPropertySourceBuilder.build(
+							event.getDataId(), event.getGroup(), fileExtension, prevNacosSource.isRefreshable());
 					target.replace(sourceName, newProperSource);
 					log.info("Replace Nacos Property Source : " + sourceName);